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

Use a consistent branch name when opening PRs #108

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Aug 8, 2023

Previously the automation included the buildpack release version in the branch name used for both the "Prepare release" and "Update builder" PRs.

For example:
prepare/v0.1.7
update/heroku/buildpacks-go/0.1.7

This meant that multiple PRs can end up being opened, for example if:

  • Someone triggers a "Prepare release" PR, but then triggers another with a different bump level (such as if they realised they had chosen the wrong version bump initially)
  • The "Update builder" PR is not merged for one release, before the next release occurs (as seen in Update heroku/buildpacks-go to v0.1.6 cnb-builder-images#371).

Now, the branch names no longer include the buildpack version, for example:
prepare-release
update/heroku/buildpacks-go

...meaning that the initial PR gets updated instead, per:
https://github.com/peter-evans/create-pull-request#action-behaviour

GUS-W-13907279.

Previously the automation included the buildpack release version in
the branch name used for both the "Prepare release" and "Update
builder" PRs.

For example:
`prepare/v0.1.7`
`update/heroku/buildpacks-go/0.1.7`

This meant that multiple PRs can end up being opened, for example if:
- Someone triggers a "Prepare release" PR, but then triggers another
  with a different bump level (such as if they realised they had chosen
  the wrong version bump initiall)
- The "Update builder" PR is not merged for one release, before the
  next release occurs (as seen in ).

Now, the branch names no longer include the buildpack version, for
example:
`prepare-release`
`update/heroku/buildpacks-go`

...meaning that the initial PR gets updated instead, per:
https://github.com/peter-evans/create-pull-request#action-behaviour
@edmorley edmorley added the enhancement New feature or request label Aug 8, 2023
@edmorley edmorley self-assigned this Aug 8, 2023
@edmorley edmorley marked this pull request as ready for review August 8, 2023 19:33
@edmorley edmorley requested a review from a team as a code owner August 8, 2023 19:33
@edmorley edmorley merged commit 544084e into main Aug 8, 2023
@edmorley edmorley deleted the edmorley/use-consistent-branch-for-prs branch August 8, 2023 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants