Uh oh!
There was an error while loading. Please reload this page.
feat: enable to check auth token to github packages - #219
Conversation
🦋 Changeset detectedLatest commit: cf0d585 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
3c43eb7 to
cf0d585Compare
iansan5653
left a comment
There was a problem hiding this comment.
We need this too -- our npmrc has a line that looks like:
//npm.pkg.github.com/:_authToken=${PACKAGES_TOKEN}Netail
commented
Jan 28, 2023
Time to merge? Note: it would be nice to create a new .npmrc file with |
claudiocantara
commented
Mar 21, 2023
Would you guys please merge this? |
JasperAlexander
commented
Mar 24, 2023
I need it too. Please merge if it works! |
MarekSyk
commented
Mar 25, 2023
via email
u nás sen říká ahoj tak ještě jednou ahoj omlouvám se, pokud se vyskytl
problém, tak se omlouvám, že mám tolik e-mailů půl vir sám mám problém s
jakou komunikací a githubem jsem požádal o pomoc a nasadil mě do tolika
vlaků, že nevím, co s tím žertem mohu pomoci tak prosím podrobný popis
toho, co jak kde rozumím těmto věcem velmi málo děkuji Marku a omlouvám se
za problém
út 21. 3. 2023 v 2:09 odesílatel Claudio Alcantara ***@***.***>
napsal: … Would you guys please merge this?
I need it too.
—
Reply to this email directly, view it on GitHub
<#219 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYJT2HCRXAD7FHXLIBP5X43W5D5VVANCNFSM6AAAAAAQ4TQGVE>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
leojuriolli7
commented
May 26, 2023
I need this as well. Would you guys please consider merging this? |
pleerock
commented
Aug 7, 2023
This one is critical to have, any estimates to accept this PR / or propose alternatives? |
peschee
commented
Aug 25, 2023
Is there any chance this gets merged? Currently, it seems that GitHub Packages as NPM registry are not supported. |
| return /^\s*\/\/registry\.npmjs\.org\/:[_-]authToken=/i.test(line); | ||
| // npm Registry: check based on https://github.com/npm/cli/blob/8f8f71e4dd5ee66b3b17888faad5a7bf6c657eed/test/lib/adduser.js#L103-L105 | ||
| // GitHub Packages: check based on https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-npm-registry#authenticating-with-a-personal-access-token | ||
| return /^\s*\/\/registry\.npmjs\.org\/:[_-]authToken=/i.test(line) || /^\s*\/\/npm\.pkg\.github\.com\/:[_-]authToken=/i.test(line) |
There was a problem hiding this comment.
Shouldn't the regex here not be registry-specific but rather look for something more generic like just the :_authToken= part?
belfz
commented
Oct 17, 2025
Any chance this will be merged soon? |
solve #178