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

fix(revert): "fix(deps): update dependency npm to v9" #561

Merged
merged 1 commit into from Jan 8, 2023
Merged

Conversation

travi
Copy link
Member

@travi travi commented Jan 8, 2023

This reverts commit 28265ec.

@travi travi requested a review from a team January 8, 2023 03:33
@travi
Copy link
Member Author

travi commented Jan 8, 2023

thankfully this didnt release because our pipeline needs the node version updated for releases, but this further highlights that the local npm version is not being used as intended (this would cause test failures for legacy auth if it were).

anyway, i dont think we would want to upgrade to non-lts npm versions even if everything were legitimately passing.

@gr2m gr2m changed the title Revert "fix(deps): update dependency npm to v9" fix(revert): "fix(deps): update dependency npm to v9" Jan 8, 2023
@travi travi merged commit c28a848 into master Jan 8, 2023
@travi travi deleted the revert-npm branch January 8, 2023 19:44
@github-actions
Copy link

🎉 This PR is included in version 10.0.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link

🎉 This PR is included in version 9.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

2 participants