Skip to content

Commit

Permalink
MINOR: [Go] [Benchmarking] fix repo specifier (#36104)
Browse files Browse the repository at this point in the history
The Arrow Conbench deployment recently imposed stricter validation, that the `repository` key must be a URL.

This PR fixes the Go benchmarking script to comply.

Authored-by: Austin Dickey <austin3spammy@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
  • Loading branch information
austin3dickey authored Jun 15, 2023
1 parent 475b5b9 commit 99dd998
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ci/scripts/go_bench_adapt.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
# Assume GitHub Actions CI. The environment variable lookups below are
# expected to fail when not running in GitHub Actions.
github_commit_info = {
"repository": os.environ["GITHUB_REPOSITORY"],
"repository": f'{os.environ["GITHUB_SERVER_URL"]}/{os.environ["GITHUB_REPOSITORY"]}',
"commit": os.environ["GITHUB_SHA"],
"pr_number": None, # implying default branch
}
Expand Down

0 comments on commit 99dd998

Please sign in to comment.