-
Notifications
You must be signed in to change notification settings - Fork 135
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
Create PAT for blocklist import/export #672
Comments
cc @nodejs/tsc |
added label since this has been open for two weeks with no review and I need +1s from the @nodejs/tsc 😅 |
+1 |
1 similar comment
+1 |
+1 |
I just tried to help @bnb by trying to create the token, but I don't have the authentication credentials for @nodejs-github-bot. Can someone else step in to do this? @nodejs/tsc @nodejs/github-bot @nodejs/build |
Sure, I can create a token for the github-bot. Its credentials are part of the secrets repo too for future ref. Do we have preferred ways of sharing such a token? |
@phillipj if you'd want to just add it to the nodejs/.github repo as a GitHub Secret so you don't need to share it, that'd be sick. Ideally, it'd be named |
@bnb sounds good to me 👍🏻 I've already created the token, so if you provide me sufficient privileges in that repo, I'll create |
I've added @phillipj as |
✅ @aduh95 feel free to remove my admin privileges. |
For nodejs/.github#1 to be able to actually be run, it will need a PAT from an account that is an admin of both organizations - meaning, nodejs/ and pkgjs/. My assumption is that if @nodejs-github-bot isn't already an admin of pkgjs/, we should trivially be able to fix that but it'll still need to be done.
As such, I'm explicitly requesting a PAT with the
admin:org
scope be created for the nodejs/.github so the import/export tooling can be run. Theadmin:org
scope is required by GitHub, and documentation for this requirement can be found here.The text was updated successfully, but these errors were encountered: