[llvm] [workflows] Fix permissions check for creating new releases (PR #81163)
Tom Stellard via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 8 14:31:46 PST 2024
================
@@ -77,20 +77,23 @@ def upload_files(repo, release, files):
parser.add_argument("--token", type=str)
parser.add_argument("--release", type=str)
parser.add_argument("--user", type=str)
+parser.add_argument("--user-token", type=str)
# Upload args
parser.add_argument("--files", nargs="+", type=str)
args = parser.parse_args()
-github = github.Github(args.token)
-llvm_org = github.get_organization("llvm")
+gh = github.Github(args.token)
+llvm_org = gh.get_organization("llvm")
llvm_repo = llvm_org.get_repo("llvm-project")
+if not args.user_token:
----------------
tstellar wrote:
Someone running locally might want to just use a single token for every API query. This is what I did for testing. Are you saying we should make the --user-token arg mandatory if the --user option is specified?
https://github.com/llvm/llvm-project/pull/81163
More information about the llvm-commits
mailing list