Skip to content

Conversation

@cometkim
Copy link
Contributor

What's the problem this PR addresses?

At #6898, I made a mistake, making it not work for scoped packages.

@arcanis
Copy link
Member

arcanis commented Sep 19, 2025

While we're at it is it possible to add tests for the feature though a mock endpoint?

@cometkim
Copy link
Contributor Author

Yes, a simple mock test can be added for confirmed behavior. But we need to check it e2e first.

cometkim added a commit to cometkim/npgi that referenced this pull request Sep 19, 2025
@cometkim
Copy link
Contributor Author

testing now..

@cometkim
Copy link
Contributor Author

cometkim commented Sep 19, 2025

@cometkim
Copy link
Contributor Author

@arcanis Can we release the patch before more people experience the issue? I can't add the test files right now. Or I'll add it maybe tomorrow.

@arcanis arcanis merged commit 7b0f301 into yarnpkg:master Sep 22, 2025
24 of 26 checks passed
@kdy1
Copy link

kdy1 commented Sep 23, 2025

Can we get this PR released? This is blocking the publish pipeline of SWC****

@arcanis
Copy link
Member

arcanis commented Sep 23, 2025

Release in progress 🚋 🚋 🚋

@arcanis
Copy link
Member

arcanis commented Sep 23, 2025

4.10.3 is available

@kdy1
Copy link

kdy1 commented Sep 23, 2025

Thank you!

cometkim added a commit to rescript-lang/rescript that referenced this pull request Sep 23, 2025
@cometkim cometkim deleted the oidc-publishing-fix branch September 25, 2025 05:20
cknitt pushed a commit to rescript-lang/rescript that referenced this pull request Sep 26, 2025
cometkim added a commit to rescript-lang/rescript that referenced this pull request Sep 26, 2025
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.

3 participants