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

Feature/reaction speed test measure #52

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

JAM-PARK
Copy link
Member

설명 (Description)

  • react-native-svg를 사용하다 라이브러리 버전 문제가 생겨 올리는 PR입니다.

관련 이슈 (Related Issues)

체크리스트 (Checklist)

  • iOS에서 테스트 완료
  • Android에서 테스트 완료
  • 웹에서 테스트 완료
  • 관련 문서 업데이트 완료 (필요시)

리뷰 가이드라인 (Review Guidelines)

  • P1 (Priority 1): 반드시 바로 수정해야 할 항목입니다. 기능에 큰 영향을 주거나, 배포 전에 꼭 해결해야 하는 중요한 사항입니다.

    • 예시: 기능, 성능 오류, 보안 취약점
  • P2 (Priority 2): 수정하면 코드가 더 개선될 수 있는 사항입니다. 큰 이슈는 아니지만, 수정하면 코드가 더 깔끔하고 유지보수가 쉬워집니다.

    • 예시: 코드 스타일, 중복, 비효율
  • P3 (Priority 3): 당장 수정하지 않아도 괜찮지만, 나중에 개선하면 도움이 되는 사항입니다. 기본적으로 모든 리뷰는 P3로 간주해주시면 됩니다.

    • 예시: 리팩토링, 주석 추가/제거, 최적화

@JAM-PARK JAM-PARK added the feat New feature or request label Nov 16, 2024
@JAM-PARK JAM-PARK self-assigned this Nov 16, 2024
@JAM-PARK JAM-PARK requested a review from a team as a code owner November 16, 2024 06:57
- react-native-css-interop update 이후 jest failed
- mocking 추가: test 에러 발생하는 useColorScheme
- react-native-svg: 14.1.0
- react-native-css-interop: ^0.0.36
- nativewind: 4.1.23(@latest)
- react-native-css-interop: ^0.1.22(@latest)
- reaction-speed-test만 [email protected] 버전 사용 중
@JAM-PARK JAM-PARK force-pushed the feature/reaction-speed-test-measure branch from 4419cf2 to 8eec855 Compare November 17, 2024 03:51
@JAM-PARK JAM-PARK marked this pull request as draft November 17, 2024 06:12
@@ -0,0 +1,7 @@
jest.mock("nativewind", () => ({
Copy link
Member

Choose a reason for hiding this comment

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

혹시 jest.setup.ts로 가져가도 동작하려나요??!

@Lim-Ji-Hyeon Lim-Ji-Hyeon marked this pull request as ready for review November 17, 2024 12:58
Copy link

@Lim-Ji-Hyeon Lim-Ji-Hyeon left a comment

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
feat New feature or request size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants