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

Yarn Berry: Ensure registry config is respected #5863

Merged
merged 1 commit into from
Oct 12, 2022

Conversation

jurre
Copy link
Member

@jurre jurre commented Oct 10, 2022

When a global/scoped registry is configured in the yarnrc, Dependabot should honor this for all requests, also for Yarn Berry, which prior to this change wasn't the case.

This ensures we correctly parse it etc.

@jurre jurre requested a review from a team as a code owner October 10, 2022 13:07
@jurre jurre force-pushed the jurre/yarn-berry-registry-finder-support branch 2 times, most recently from f482423 to f31e883 Compare October 10, 2022 13:39
@jurre jurre force-pushed the jurre/yarn-berry-registry-finder-support branch 7 times, most recently from 6e60d0c to f65b782 Compare October 11, 2022 12:27
@yarinsa
Copy link

yarinsa commented Oct 11, 2022

+1 Thanks very helpful!

How did you figure out the npmAuthIdent?

#5870

When a global/scoped registry is configured in the yarnrc, Dependabot should
honor this for all requests, also for Yarn Berry, which prior to this
change wasn't the case.

This ensures we correctly parse it etc.
@jurre jurre force-pushed the jurre/yarn-berry-registry-finder-support branch from f65b782 to 88f5ddf Compare October 11, 2022 18:07
@jurre
Copy link
Member Author

jurre commented Oct 12, 2022

+1 Thanks very helpful!

How did you figure out the npmAuthIdent?

#5870

I don't think it's related? Let's debug your problem in the issue you've opened for it though 👍

@jurre jurre merged commit cbd0875 into main Oct 12, 2022
@jurre jurre deleted the jurre/yarn-berry-registry-finder-support branch October 12, 2022 10:34
@pavera pavera mentioned this pull request Oct 31, 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.

None yet

5 participants