SpecifyAnimationSpecIssue 구현 #206

limsaehyun · 2022-10-05T09:42:53Z

Issue

close Develop lint for internal usage #153

작업 내용

SpecifyAnimationSpecIssue 생성
SpecifyAnimationSpecTest 작성
SpecifyAnimationSpecDetector 구현

github-actions · 2022-10-05T09:49:19Z

❌ Android CI failed:

jisungbin · 2022-10-05T09:52:41Z

lint-compose/src/test/kotlin/team/duckie/quackquack/lint/compose/SpecifyAnimationSpecTest.kt

+ * 2. animate*AsState 함수를 대상으로 경고해야 함 [AnimateStateOfList]
+ * 2. argument animationSpec 에 quackAnimationSpec 을 사용해야 함
+ */
+class SpecifyAnimationSpecTest {


AnimateStateOfList 에서 animationSpec 인자 정의가 빠져서 default arguments 가 적용된 경우에도 SpecifyAnimationSpec 린트가 적용되는지 테스트 해봐야 할 거 같아요!

default argument를 사용 할 경우(animationSpec 인자를 찾지 못할 경우) 에러를 방출하도록 수정하였습니다

jisungbin · 2022-10-05T09:59:03Z

CI Fails in `QuackBorder.kt:77`

See: #207

jisungbin · 2022-10-05T09:59:29Z

develop 브런치로 잘 해주셨네요! 슬랙 공지 확인 감사합니다 ㅎㅎ!

github-actions · 2022-10-05T10:04:00Z

❌ Android CI failed:

jisungbin

감사합니다! 수고하셨어요 ㅠㅠ

jisungbin · 2022-10-05T11:20:32Z

...-compose/src/main/kotlin/team/duckie/quackquack/lint/compose/SpecifyAnimationSpecDetector.kt

+                    }
+                }
+
+                sendErrorReport(node.sourcePsi)


와 세현님 구현 천재시네요 ㅋㅋㅋ

헉..... 감사합니다아 🥰

jisungbin · 2022-10-05T11:21:46Z

/automerge

ㄹㅇ 필요,,, 만들어야겠어요 빨리
CI 통과되면 그냥 머지해주셔도 됨니당

See: #208

limsaehyun · 2022-10-05T11:23:25Z

있으면 진짜 진짜 편하긴 하겠네요
근데 아마 위 이슈때문에 CI는 블락 당할 것 같아요 ㅜ.ㅜ

jisungbin · 2022-10-05T11:24:09Z

있으면 진짜 진짜 편하긴 하겠네요 근데 아마 위 이슈때문에 CI는 블락 당할 것 같아요 ㅜ.ㅜ

아 맞다! 그럼 걍 머지 하죠 ㅋㅋ 고고씽

github-actions · 2022-10-05T11:27:10Z

❌ Android CI failed:

* refactor: optimized svg assets (#189) * refactor: env 광역으로 이동 (#191) env 광역으로 이동 및 키 재지정 (조직 secrets) * [CI] bump versions * feat: committer bot 설정 (#193) * refactor: set committer bot account * chore(deps): update kover to v0.6.1 (#202) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * docs: copyright 최신화 (#204) * docs: update copyright * comment: 레포지토리 이동에 따른 copyright 변경 * refactor: 꽥꽥 로고에 배경색 추가 * Lint/specify animation spec (#206) * feat: SpecifyAnimationSpecTest 작성 * feat: SpecifyAnimationSpecDetector 구현 * feat: SpecifyAnimationSpecIssue 추가 * feat: LongMethod suppress * feat: default argument 사용 시 에러 * fix(deps): update dependency com.google.testparameterinjector:test-parameter-injector to v1.9 (#210) * fix(deps): update dependency com.google.testparameterinjector:test-parameter-injector to v1.9 Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore: bump kotlin 1.7.20 (#211) * fix(deps): update dependency androidx.compose.material3:material3 to v1.0.0-rc01 (#213) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * feat: ux-writing-overlay 구현 시도 (#223) * feat: animateAsState 빌드 에러 해결 (#224) * chore: bump AGP to 8.0.0-alpha03 * refactor: animationSpec 으로 quackAnimationSpec() 사용하도록 설정 * chore: fix build error (#226) * refactor: suppress UNUSED_VARIABLE * refactor: remove wildcard from test naming * refactor: bump public apis * refactor: move magic-number to global variable * refactor: 과한 @composable 제거 - 단순 연산보다 @composable 을 정의하는 비용이 더 비쌀 것으로 예상됨 * feat: 스냅샷 테스트 복원 (#231) * feat: 스냅샷 테스트 복원 * feat: 스냅샷 테스트 복원에 따른 QuackAnimationSpec() 변경 대응 * feat: 빠진 animateValueAsState 조건 추가 * chore: 스냅샷 테스트 모듈 활성화 * chore: 사용하지 않는 의존성 주석 제거 * chore: bump public apis * refactor: 불필요한 타입 명시 제거 * refactor: 인자에 new-line 추가 * test: TODO 테스트에 ignore 추가 * chore: glide -> coil * chore: glide -> coil * feat: snapshots 폴더 gitignore 추가 * chore(deps): update ksp to v1.7.20-1.0.6 (#227) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * fix(deps): update dependency androidx.datastore:datastore-preferences to v1.1.0-dev01 (#220) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Revert "fix(deps): update dependency androidx.datastore:datastore-preferences to v1.1.0-dev01 (#220)" (#233) This reverts commit efa4731. * refactor: ui-components variant 모듈 네이밍 방식 통일 (#234) * refactor: QuackFontScale 제거 (#235) QuackFontScale 에서 Density 로 변경 * feat: QuackIcon 표시 방법 변경 (#236) * feat: QuackIcon 은 Modifier#paint 로 그리게 변경 * feat: QuackColor 에 Transparent 추가 * refactor: toggle 플레이그라운드 네이밍 통일 - QuackToggle -> QuackIconToggle * feat: changed api dump * Feat/windowinsets (#237) * feat: QuackIcon 은 Modifier#paint 로 그리게 변경 * feat: QuackColor 에 Transparent 추가 * refactor: toggle 플레이그라운드 네이밍 통일 - QuackToggle -> QuackIconToggle * feat: changed api dump * feat: WindowInsets 이 작동하지 않던 버그 수정 * feat: remove needless newline * feat: QuackLargeButton 텍스트 start 패딩에 4dp 추가 (#238) * feat: snapshots 관련 CI 및 로직 추가 (#241) - gradle scan 라이선스 동의 로직 추가 - paparazzi 복구 - 스냅샷 검증 CI 추가 * refactor: 스냅샷 CI job 이름 변경 * feat: snapshot utils (#243) * refactor: AnimationTestRule 코드 스타일 정리 * feat: paparazzi 디바이스 옵션 추가 * feat: fix snapshot actions (#244) * refactor: AnimationTestRule 코드 스타일 정리 * feat: paparazzi 디바이스 옵션 추가 * refactor: if 위치 변경 * refactor: 사용하지 않는 테스트 인자 제거 * feat: 잘못된 if 구문 정정 * QuackBottomSheet 등 여러가지 컴포넌트 구현 (#217) * chore: indentation 4로 설정 * feat: label 추가 * feat: WIP bottom sheet 컴포넌트 추가 * feat: dropDown 컴포넌트 추가 * feat: selectableImage 컴포넌트 추가 * chore: indentation 4로 설정 * feat: label 추가 * feat: WIP bottom sheet 컴포넌트 추가 * feat: dropDown 컴포넌트 추가 * feat: selectableImage 컴포넌트 추가 * chore: INTERNET PERMISSION 추가 * feat: SelectableImage QuackIcon 추가 * feat: Black80, Gray6 추가 * feat: image parameter rippleEnabled 추가하고 tint 변경 * feat: quackBorderOrNull extension Modifier 생성 * fix: deleteBg 사이즈 변경 * feat: SelectableImage playGround에 추가 * fix: DeleteBg 네이밍 변경 * feat: SelectableImage Kdoc 추가 및 세세하게 파라미터 변경 * fix: arrown down size 변경 * feat: EtcPlayground에 Label, Dropdown 넣어놓기 * feat: SkyblueColor 추가 * fix: clickable 위치 변경해서 ripple 모양 바꾸기 * feat: Label, Dropdown 문서 정리 * feat: QuackTopAppBar 추가 * feat: QuackTopAppBar 문서화하고 parameter 정리 * docs: QuackTopAppBar checkValidation 함수 문서화 * feat: QuackBottomNavigation 구현 및 문서화 * chore: ktlint 맞추기 * refac: ktlint 맞추기 * chore: ktlin 맞추기 * chore: ktlint 적용 * test: paparazzi 테스트 코드 추가 * fix: Typography onClick 수정 * feat: QuackBottomSheet 구현 눈물을 머금고 material로 완성 * feat: QuackBottomSheet OnClick 추가 및 필요한 코드 추가 * feat: typography에 singleLine parameter 추가 * feat: QuackRoundImage 구현 * feat: Transparent 색깔 추가 * refac: QuackBottomSheet Component 및 Playground 코드 정리 * fix: BottomNavigationItem Icon 변경 * refac: ktlint 적용 * refac: lint 적용 * docs: 구현한 컴포넌트 벤치마크 추가 * fix: SelectedFilterBox BoxScope으로 다시 변경 Co-authored-by: Ji Sungbin <ji@sungb.in> * QuackModalDrawer, Divider 구현 (#215) * feat: Divder 구현 * feat: Divider 반환 타입 설정 * feat: 드래그 가능한 drawer까지 구현 * feat: BlackOpacity60 Color 추가 * feat: Drawer 외부 Opacity 설정 * docs: Annotation 상단으로 끌어올림 * feat: ModalDrawer Playground 추가 * feat: Code 정리와 Preview 제거 * Revert "feat: ModalDrawer Playground 추가" This reverts commit ff587a4. * Revert "Revert "feat: ModalDrawer Playground 추가"" This reverts commit ebcf356. * feat: Width가 Infinite일때 예외 처리 및 confirmStateChange 추가 * feat: Content가 Modal 내부에 받아지도록 해야함 * refactor: 하드코딩 제거 * feat: QuackModalDrawerImpl 패키지 옮기기 * refactor: 패키지 정리 * refactor: unused import * docs: ModalDrawerImpl Kdoc * docs: drawer Kdoc 작성 * remove: unused import * refactor: public 메서드와 , 추가 * chore: dump api * remove: empty line * feat: Benchmark 추가 * feat: benchmark에 의존성 추가 * refactor: 애니메이션 스펙 네이밍 변경 적용 Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Choi Sang Rok <tkdfhr7497@naver.com> Co-authored-by: 임세현 <80076029+limsaehyun@users.noreply.github.com> Co-authored-by: goddoro <dongsang1222@daum.net>

limsaehyun added 3 commits October 5, 2022 14:07

feat: SpecifyAnimationSpecTest 작성

b9a1f8b

feat: SpecifyAnimationSpecDetector 구현

7677eb7

feat: SpecifyAnimationSpecIssue 추가

2878271

limsaehyun added lint documentation Rebuild project website labels Oct 5, 2022

limsaehyun requested review from riflockle7 and jisungbin October 5, 2022 09:42

limsaehyun self-assigned this Oct 5, 2022

feat: LongMethod suppress

9402bda

jisungbin suggested changes Oct 5, 2022

View reviewed changes

feat: default argument 사용 시 에러

88841c4

jisungbin approved these changes Oct 5, 2022

View reviewed changes

jisungbin merged commit 4be7b88 into develop Oct 5, 2022

jisungbin deleted the lint/SpecifyAnimationSpec branch October 5, 2022 11:24

jisungbin changed the title ~~Lint/specify animation spec~~ SpecifyAnimationSpecIssue 구현 Oct 5, 2022

jisungbin mentioned this pull request Oct 5, 2022

Develop lint for internal usage #153

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

SpecifyAnimationSpecIssue 구현 #206

SpecifyAnimationSpecIssue 구현 #206

limsaehyun commented Oct 5, 2022

github-actions bot commented Oct 5, 2022

jisungbin Oct 5, 2022

limsaehyun Oct 5, 2022 •

edited

Loading

jisungbin commented Oct 5, 2022 •

edited

Loading

jisungbin commented Oct 5, 2022

github-actions bot commented Oct 5, 2022

jisungbin left a comment

jisungbin Oct 5, 2022

limsaehyun Oct 5, 2022

jisungbin commented Oct 5, 2022 •

edited

Loading

limsaehyun commented Oct 5, 2022

jisungbin commented Oct 5, 2022 •

edited

Loading

github-actions bot commented Oct 5, 2022

SpecifyAnimationSpecIssue 구현 #206

SpecifyAnimationSpecIssue 구현 #206

Conversation

limsaehyun commented Oct 5, 2022

Issue

작업 내용

github-actions bot commented Oct 5, 2022

jisungbin Oct 5, 2022

Choose a reason for hiding this comment

limsaehyun Oct 5, 2022 • edited Loading

Choose a reason for hiding this comment

jisungbin commented Oct 5, 2022 • edited Loading

CI Fails in QuackBorder.kt:77

jisungbin commented Oct 5, 2022

github-actions bot commented Oct 5, 2022

jisungbin left a comment

Choose a reason for hiding this comment

jisungbin Oct 5, 2022

Choose a reason for hiding this comment

limsaehyun Oct 5, 2022

Choose a reason for hiding this comment

jisungbin commented Oct 5, 2022 • edited Loading

limsaehyun commented Oct 5, 2022

jisungbin commented Oct 5, 2022 • edited Loading

github-actions bot commented Oct 5, 2022

limsaehyun Oct 5, 2022 •

edited

Loading

jisungbin commented Oct 5, 2022 •

edited

Loading

CI Fails in `QuackBorder.kt:77`

jisungbin commented Oct 5, 2022 •

edited

Loading

jisungbin commented Oct 5, 2022 •

edited

Loading