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

디펜던시 정리 #249

Merged
merged 1 commit into from
Oct 3, 2023
Merged

디펜던시 정리 #249

merged 1 commit into from
Oct 3, 2023

Conversation

tirr-c
Copy link
Contributor

@tirr-c tirr-c commented Sep 28, 2023

  • nyc 삭제. istanbul을 통해 코드 커버리지를 측정하는 커맨드라인 툴인데, 지금 커버리지를 측정해서 하고 있는 게 없기 때문에 우선 삭제했습니다. 요즘 커버리지 측정 방식은 조금 다른 것 같은데 조사가 필요합니다.
  • ioredis 4 → 5로 업데이트. 마이그레이션 가이드를 읽어 봤는데, 타입스크립트 정의를 기본 제공한다는 점 빼고는 신경 쓸 게 없어 보였습니다. 타입 정의가 살짝 변해서 관련 변경사항이 src/oidc/redis.ts에 있습니다.
  • argon2의 디펜던시에 node-gyp 추가. 없으면 애플 실리콘 맥에서 터집니다.
  • make-dir@^3.1.0을 강제로 4.0.0으로 매핑. 최소 Node.js 버전이 10으로 올라갔다는 것 외에 별 변경사항은 없습니다. 이렇게 하면 semver 버전을 한 종류로 맞출 수 있게 됩니다.

@tirr-c tirr-c self-assigned this Sep 28, 2023
Copy link
Contributor

@seojangho seojangho left a comment

Choose a reason for hiding this comment

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

LGTM.

@tirr-c tirr-c merged commit 45e5285 into master Oct 3, 2023
2 checks passed
@tirr-c tirr-c deleted the clean-up-deps branch October 3, 2023 07:24
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