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

chore: don't have mergify delete branches #10264

Merged
merged 1 commit into from
Sep 9, 2020
Merged

Conversation

rix0rrr
Copy link
Contributor

@rix0rrr rix0rrr commented Sep 9, 2020

Branches are automatically deleted by GitHub itself, so Mergify doesn't
have to do that.

In fact, Mergify trying to do it leaves the end result of every
closed PR with a failing "cross" icon because the Mergify action
to delete the head branch fails.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Branches are automatically deleted by GitHub itself, so Mergify doesn't
have to do that.

In fact, Mergify trying to do it leaves the end result of every
closed PR with a failing "cross" icon because the Mergify action
to delete the head branch fails.
@rix0rrr rix0rrr requested a review from a team September 9, 2020 12:03
@rix0rrr rix0rrr self-assigned this Sep 9, 2020
@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject6AEA49D1-qxepHUsryhcu
  • Commit ID: 4d1782e
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

nija-at
nija-at previously requested changes Sep 9, 2020
Copy link
Contributor

@nija-at nija-at left a comment

Choose a reason for hiding this comment

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

This seems useful to clean up branches from the gazillion PRs we have.

If we don't like the way mergify does it, can we get GH actions to do it?

@nija-at
Copy link
Contributor

nija-at commented Sep 9, 2020

Just saw your comment in the chat about GH doing this automatically. If that's so, I'm ok with this change.

@rix0rrr
Copy link
Contributor Author

rix0rrr commented Sep 9, 2020

I said that in the PR description as well.

@rix0rrr rix0rrr merged commit effceb5 into master Sep 9, 2020
@rix0rrr rix0rrr deleted the huijbers/no-delete-head branch September 9, 2020 13:46
@nija-at
Copy link
Contributor

nija-at commented Sep 9, 2020

Sorry :(

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.

4 participants