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

Add auth setup for private registry with scope #16

Conversation

vitoorgomes
Copy link
Contributor

@vitoorgomes vitoorgomes commented Sep 9, 2023

Bun is awesome, but I've seen too many workarounds to make authentication for a private package under a registry-url (like 'https://npm.pkg.github.com/') with a scope

Heavily inspired by the setup-node action, I created this to support this feature

I've tested a lot on a private personal org with a private package and my repo that installs the private package on CI worked like a charm.

image

SCR-20230909-6r

Also "@actions/cache" import was duplicated on setup.ts and now I left only import * as cache from "@actions/cache";

Copy link

@zerocowl zerocowl left a comment

Choose a reason for hiding this comment

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

@vitoorgomes Nice job! I also have some suggestions

src/auth.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@CyberFlameGO CyberFlameGO left a comment

Choose a reason for hiding this comment

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

Might wanna remove node_modules - generally this shouldn't be committed

@xhyrom
Copy link
Collaborator

xhyrom commented Sep 26, 2023

There are some conflicts, can you resolve them?

@vitoorgomes vitoorgomes force-pushed the feature/add-authentication-setup-to-private-registries-with-scope branch from d3f5622 to 3f00658 Compare October 17, 2023 18:46
@vitoorgomes
Copy link
Contributor Author

Might wanna remove node_modules - generally this shouldn't be committed

when I forked node_modules were still a thing in this repo, gladly it was removed I also removed from this PR

@vitoorgomes
Copy link
Contributor Author

vitoorgomes commented Oct 17, 2023

There are some conflicts, can you resolve them?

didn't had free time sooner, but it's done

@xhyrom
Copy link
Collaborator

xhyrom commented Oct 25, 2023

can you rebuild it one more time, please? Then ping me, i'll merge.

@vitoorgomes vitoorgomes force-pushed the feature/add-authentication-setup-to-private-registries-with-scope branch from 3f00658 to 345d8f9 Compare October 25, 2023 20:19
@vitoorgomes
Copy link
Contributor Author

can you rebuild it one more time, please? Then ping me, i'll merge.

sure, done

@xhyrom xhyrom merged commit 830e319 into oven-sh:main Nov 1, 2023
8 of 10 checks passed
@vitoorgomes vitoorgomes deleted the feature/add-authentication-setup-to-private-registries-with-scope branch November 16, 2023 19:20
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