[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