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: downgrade @pnpm/npm-conf to remove annoying builtin warning #5521

Merged
merged 1 commit into from Oct 19, 2022

Conversation

lvqq
Copy link
Member

@lvqq lvqq commented Oct 19, 2022

Close #5518

The warning is a little annoying. It will be removed in the next patch version and be replaced in anthoer way in the future.

@lvqq lvqq requested a review from zkochan as a code owner October 19, 2022 13:29
@zkochan
Copy link
Member

zkochan commented Oct 19, 2022

It is annoying but otherwise it causes no issues. I think it is OK to implement the new pnpm doctor command as it shouldn't be hard.

@lvqq
Copy link
Member Author

lvqq commented Oct 19, 2022

Yes, it's not hard and I will implement it in the next days. I just want to add a quick fix to remove the warning as there are many people being confused about it.

@zkochan zkochan merged commit e7fd8a8 into pnpm:main Oct 19, 2022
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.

Warn Load nom builtin configs failed.
2 participants