Skip to content

chore(release): pass missing --remote argument to release workflows#3883

Merged
rickeylev merged 1 commit into
bazel-contrib:mainfrom
rickeylev:fix-release-workflow-args
Jul 2, 2026
Merged

chore(release): pass missing --remote argument to release workflows#3883
rickeylev merged 1 commit into
bazel-contrib:mainfrom
rickeylev:fix-release-workflow-args

Conversation

@rickeylev

Copy link
Copy Markdown
Collaborator

Workflows using process-backports and promote-rc subcommands were failing
due to missing or hardcoded remote arguments.

  • Added --remote parameter to promote-rc subcommand.
  • Updated release_process_backports.yaml and release_promote_rc.yaml to
    pass --remote origin.
  • Updated tests in release_test.py to support and verify the new argument.

Workflows using `process-backports` and `promote-rc` subcommands were failing
due to missing or hardcoded remote arguments.

- Added `--remote` parameter to `promote-rc` subcommand.
- Updated `release_process_backports.yaml` and `release_promote_rc.yaml` to
  pass `--remote origin`.
- Updated tests in `release_test.py` to support and verify the new argument.
@rickeylev rickeylev requested a review from aignas as a code owner July 2, 2026 07:49
@rickeylev rickeylev changed the title fix(release): pass missing --remote argument chore(release): pass missing --remote argument to release workflows Jul 2, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the promote-rc subcommand to accept a --remote command-line argument instead of hardcoding the git remote to 'upstream', and updates the corresponding unit tests. Feedback suggests making --remote optional with a default value of 'upstream' to maintain backwards compatibility, and updating the dry-run output message to use the configured remote instead of hardcoding 'upstream'.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread tools/private/release/promote_rc.py
# Tag the specific commit without checkout, and push to remote
self.git.tag(version, commit_sha)
self.git.push("upstream", version)
self.git.push(args.remote, version)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

While updating the push command to use args.remote instead of the hardcoded 'upstream', please also update the dry-run message on line 79 (print(f"[DRY RUN] Would push tag {version} to upstream")) to use args.remote as well, ensuring consistency between dry-run output and actual execution.

@rickeylev rickeylev merged commit 6570a9e into bazel-contrib:main Jul 2, 2026
6 of 8 checks passed
@rickeylev rickeylev deleted the fix-release-workflow-args branch July 2, 2026 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant