Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DesignSystem 린트 데모 #16

Merged
merged 17 commits into from
Aug 17, 2022
Merged

DesignSystem 린트 데모 #16

merged 17 commits into from
Aug 17, 2022

Conversation

jisungbin
Copy link
Contributor

@jisungbin jisungbin commented Aug 16, 2022

What's new?

  1. 린트 플러그인 추가
  2. DesignSystem 린트 데모 구현
  3. DesignSystem 린트 테스트 코드 추가

@jisungbin jisungbin added the lint label Aug 16, 2022
@jisungbin jisungbin self-assigned this Aug 16, 2022
@jisungbin jisungbin changed the title 컴포즈 린트 데모 구현 DesignSystem 린트 데모 Aug 16, 2022
@jisungbin jisungbin changed the title DesignSystem 린트 데모 DesignSystem 린트 데모 Aug 16, 2022
@jisungbin jisungbin merged commit 6060719 into develop Aug 17, 2022
@jisungbin jisungbin deleted the lint-compose branch August 17, 2022 11:41
Comment on lines +24 to +26
applyPlugins(
PluginEnum.Jacoco,
)
Copy link
Contributor

@riflockle7 riflockle7 Aug 17, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

applyPlugins(
    plugins = arrayOf(PluginEnum.Jacoco),
)

이렇게 하면 default 파라미터 넣을 수 있는데 이건 너무 짜치는 거겠죠? 🤣

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오 헐 저렇게 할 수 있군요!! 근데 그레이들까지 코드 규칙 적용하기엔 너무 못생겨서져서 그레이들은 예외로 하는걸로 말씀 드렸어용 어제 회의때!

Copy link
Contributor

@riflockle7 riflockle7 Aug 17, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

varargs 는 array 로 인식되어서 저렇게 할 수 있더라구요 +_+

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

신기하네용

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants