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

[mergify] automerge PRs for mergify prs with auto-bumps #25632

Merged
merged 2 commits into from
May 10, 2021

Conversation

v1v
Copy link
Member

@v1v v1v commented May 10, 2021

What does this PR do?

Enable auto-merge when the branch protection requires a review. See review example in mergify.

The suggested approach was also to enable mergify in the list of users to merge -> https://docs.mergify.io/faq/#mergify-is-unable-to-merge-my-pull-request-due-to-my-branch-protection-settings. But it might somehow the other users to be restricted, so let's avoid that particular case

Why is it important?

Otherwise the bump update automation will require manual actions anyway

@v1v v1v added automation Team:Automation Label for the Observability productivity team labels May 10, 2021
@v1v v1v requested review from jsoriano and a team May 10, 2021 10:46
@v1v v1v self-assigned this May 10, 2021
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels May 10, 2021
@elasticmachine
Copy link
Collaborator

elasticmachine commented May 10, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #25632 updated

  • Start Time: 2021-05-10T11:22:20.469+0000

  • Duration: 17 min 32 sec

  • Commit: 834f0f1

Trends 🧪

Image of Build Times

❕ Flaky test report

No test was executed to be analysed.

Copy link
Member

@jsoriano jsoriano left a comment

Choose a reason for hiding this comment

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

Thanks!

@v1v v1v merged commit 900c453 into elastic:master May 10, 2021
@v1v v1v deleted the feature/mergify-auto-merge-prs-bump branch May 10, 2021 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants