Skip to content
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

A couple of fixes to cherry-picking and make_pr.sh #19322

Merged
merged 5 commits into from
Jun 15, 2023
Merged

Conversation

thejcannon
Copy link
Member

@thejcannon thejcannon commented Jun 15, 2023

  • Use jq -r everywhere to get raw strings, not quoted ones
  • Don't add a space before putting the author as a reviewer in REVIEWERS so that the space replacement will work correctly
  • The right flag is --reviewer not --reviewers
  • Plumbing to allow the cherry picker to pick changes to make_pr.sh (namely switching the github action back to using main after pushing the branch, and plumbing that through make_pr.sh).

@thejcannon thejcannon added the category:internal CI, fixes for not-yet-released features, etc. label Jun 15, 2023
@thejcannon thejcannon requested a review from huonw June 15, 2023 02:21
@thejcannon thejcannon changed the title Fix make_pr.sh A couple of fixes to make_pr.sh Jun 15, 2023
@thejcannon thejcannon changed the title A couple of fixes to make_pr.sh A couple of fixes to cherry-picking and make_pr.sh Jun 15, 2023
build-support/cherry_pick/make_pr.sh Outdated Show resolved Hide resolved
@thejcannon thejcannon enabled auto-merge (squash) June 15, 2023 11:44
@thejcannon thejcannon merged commit 68be3f0 into main Jun 15, 2023
@thejcannon thejcannon deleted the jcannon/fix_make_pr branch June 15, 2023 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:internal CI, fixes for not-yet-released features, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants