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.

Sign up for GitHub

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

Jump to bottom

[mergify] change title for the backports #25396

Merged
merged 1 commit into from
Apr 29, 2021
Merged

[mergify] change title for the backports #25396

merged 1 commit into from
Apr 29, 2021

Conversation

v1v
Copy link
Member

@v1v v1v commented Apr 28, 2021

What does this PR do?

Since Mergifyio/mergify#2419 has been solved we can now change the title.

This is a proposal to change the title in the PRs that are backported with mergify.

[target-branch](backport #1234) title

Why is it important?

Highlight what's important in the title to easily filter target branches

Task

  • If we agree then we need to change the script in charge to generate the changelog.

@v1v v1v added the Team:Automation Label for the Observability productivity team label Apr 28, 2021
@v1v v1v requested review from jsoriano, urso, ruflin and a team April 28, 2021 16:21
@v1v v1v self-assigned this Apr 28, 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 Apr 28, 2021
@v1v v1v mentioned this pull request Apr 28, 2021
Merged
@v1v v1v changed the title [mergify]: change title [mergify] change title for the backports Apr 28, 2021
@elasticmachine
Copy link
Collaborator

💚 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 #25396 opened

  • Start Time: 2021-04-28T16:22:06.523+0000

  • Duration: 19 min 46 sec

  • Commit: 1c97eaa

Trends 🧪

Image of Build Times

❕ Flaky test report

No test was executed to be analysed.

urso
urso approved these changes Apr 28, 2021
View reviewed changes
Copy link

@urso urso left a comment

Choose a reason for hiding this comment

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

I don't mind. This pattern is just as easy to parse.

@v1v v1v merged commit b68dcb0 into elastic:master Apr 29, 2021
@v1v v1v mentioned this pull request Jul 1, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kuisathaverat kuisathaverat kuisathaverat approved these changes

@mdelapenya mdelapenya mdelapenya approved these changes

@ruflin ruflin ruflin approved these changes

@urso urso urso approved these changes

@jsoriano jsoriano jsoriano approved these changes

Assignees

@v1v v1v

Labels
Team:Automation Label for the Observability productivity team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@v1v @elasticmachine @jsoriano @urso @ruflin @mdelapenya @kuisathaverat