Skip to content

Conversation

pudovmaxim
Copy link
Contributor

What's the problem this PR addresses?

This fix of issue described here #6880

How did you fix it?

Fix validation of hash at explain peer-requirements command

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.

Copy link

@droddy droddy left a comment

Choose a reason for hiding this comment

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

Do we know that all peer dependency hashes will be exactly 6 characters after 'p' now?
Is this likely to break again if/when the length of that particular has changes again?

@sekyungk
Copy link
Contributor

I am the author of #6829. I didn't catch this issue. Since yarn explain is unusable, I hope it can be fixed quickly. Let me mention @arcanis , who is the reviewer of #6829. Thank you.

@arcanis arcanis merged commit b553533 into yarnpkg:master Aug 26, 2025
2 of 3 checks passed
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.

4 participants