Skip to content

docs: Clarify YN0020 #6272

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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

docs: Clarify YN0020 #6272

wants to merge 2 commits into from

Conversation

RDIL
Copy link
Member

@RDIL RDIL commented May 3, 2024

What's the problem this PR addresses?

This has been a problem that a LOT of people have had and this PR aims to make it easier to find the solution to this issue.

See also https://discord.com/channels/226791405589233664/1235967197168537630

How did you fix it?

Updated docs

Checklist

  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@@ -186,10 +186,16 @@ Running `yarn cache clean` will cause Yarn to remove everything inside `.yarn/ca

## YN0020 - `MISSING_LOCKFILE_ENTRY`

Also known as "This package doesn't seem to be present in your lockfile; run "yarn install" to update the lockfile".
Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I know, bad practice to restate the error message. I did this here because that way, search engines will show this higher in the results for the exact error message, instead of just the error code.

@RDIL RDIL added the documentation This issue or pull request is about the documentation label May 3, 2024
@arcanis
Copy link
Member

arcanis commented May 6, 2024

Can we instead make the error message clearer in the CLI itself? Perhaps by listing the path where the top-level project was found using the ReportError API?

@RDIL
Copy link
Member Author

RDIL commented May 6, 2024

Sure, will do that.

@RDIL RDIL marked this pull request as draft May 6, 2024 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This issue or pull request is about the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants