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

npm 버전 8.5이상으로 강제하고 node version을 17.7.0으로 변경합니다. #2004

Merged
merged 4 commits into from Mar 10, 2022

Conversation

pgs787
Copy link
Contributor

@pgs787 pgs787 commented Mar 10, 2022

PR 설명

Closes #1719

#1719 에 해당하는 이슈입니다.
lerna/lerna#2891 (comment)
npm 7.20 - 8.4 버전까지 TF 버전 업 시 package-lock.json가 업데이트 되지 않는 이슈가 있었습니다.
8.5이상 버전부터는 릴리즈 노트 내용과 같이 fix 되었습니다.


node 버전 업 (노드 문서)
npm version 8.5.0 이상을 지원하는 노드 버전이 17.6부터여서 노드 버전도 올리게 됐습니다.

위의 내용에 따라 npm 버전을 8.5이상으로 강제하여 이슈를 해결하고자 합니다.

변경 내역

체크리스트

스크린샷 & URL

@pgs787 pgs787 requested a review from a team as a code owner March 10, 2022 07:28
@pgs787 pgs787 self-assigned this Mar 10, 2022
inbeom
inbeom previously approved these changes Mar 10, 2022
Copy link
Contributor

@inbeom inbeom left a comment

Choose a reason for hiding this comment

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

👍

@pgs787
Copy link
Contributor Author

pgs787 commented Mar 10, 2022

헉 모르고 테스트 릴리즈를 같이 포함시켜 버렸네요 .. ㅋㅋ 다시 푸쉬하겠습니다!

@pgs787 pgs787 requested review from inbeom and removed request for inbeom March 10, 2022 07:33
inbeom
inbeom previously approved these changes Mar 10, 2022
Copy link
Contributor

@inbeom inbeom left a comment

Choose a reason for hiding this comment

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

ㅋㅋ 감사합니다!

polysiya
polysiya previously approved these changes Mar 10, 2022
Copy link
Contributor

@polysiya polysiya left a comment

Choose a reason for hiding this comment

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

👍

zhsks528
zhsks528 previously approved these changes Mar 10, 2022
Copy link
Contributor

@zhsks528 zhsks528 left a comment

Choose a reason for hiding this comment

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

👏

@pgs787 pgs787 changed the title npm 버전 8.5이상으로 강제합니다. npm 버전 8.5이상으로 강제하고 node version을 17.6.x으로 변경합니다. Mar 10, 2022
@pgs787 pgs787 requested review from inbeom, polysiya, zhsks528 and a team and removed request for giwan-dev, dddeok, inbeom, polysiya, zhsks528 and a team March 10, 2022 07:51
@pgs787 pgs787 dismissed stale reviews from zhsks528 and polysiya via 3987283 March 10, 2022 07:53
@pgs787
Copy link
Contributor Author

pgs787 commented Mar 10, 2022

�node 17.6 버전 이상부터 npm 8.5를 지원해서 노드버전도 변경했습니다!

@codecov
Copy link

codecov bot commented Mar 10, 2022

Codecov Report

Merging #2004 (dbcd07d) into main (102d161) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2004   +/-   ##
=======================================
  Coverage   30.61%   30.61%           
=======================================
  Files         545      545           
  Lines       10272    10272           
  Branches     3059     3059           
=======================================
  Hits         3145     3145           
  Misses       6857     6857           
  Partials      270      270           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 102d161...dbcd07d. Read the comment docs.

@pgs787 pgs787 added this to the v7 milestone Mar 10, 2022
@pgs787 pgs787 added help wanted Extra attention is needed and removed help wanted Extra attention is needed labels Mar 10, 2022
@@ -11,7 +11,7 @@ on:
env:
GITHUB_API_URL_BASE: https://api.github.com/repos/${{ github.repository }}
# Node.js
NODE_VERSION: '16.x'
NODE_VERSION: '17.6.x'
Copy link
Contributor

Choose a reason for hiding this comment

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

완전 블리딩엣지군요. 오늘 기준 current는 17.7.0이긴 하네요 https://nodejs.org/en/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앗 그럼 17.7.0로 적용시키도록 하겠습니다! 🙇

@pgs787 pgs787 requested review from inbeom and a team March 10, 2022 08:06
@pgs787 pgs787 changed the title npm 버전 8.5이상으로 강제하고 node version을 17.6.x으로 변경합니다. npm 버전 8.5이상으로 강제하고 node version을 17.7.0으로 변경합니다. Mar 10, 2022
@pgs787 pgs787 merged commit bad74d2 into main Mar 10, 2022
@pgs787 pgs787 deleted the npm-version-engines branch March 10, 2022 08:29
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.

버전을 올리고 npm install이 오류를 내는 문제
4 participants