Skip to content

[CI] Move PR skippable changes to pr-bot config#132461

Merged
brianseeders merged 1 commit intoelastic:mainfrom
brianseeders:move-skip-config
May 18, 2022
Merged

[CI] Move PR skippable changes to pr-bot config#132461
brianseeders merged 1 commit intoelastic:mainfrom
brianseeders:move-skip-config

Conversation

@brianseeders
Copy link
Copy Markdown
Contributor

@brianseeders brianseeders commented May 18, 2022

Support for skippable changes was just added to the PR bot, so I'm moving these regexes to the PR config.

The main benefit is that for skippable PRs, buildkite won't be triggered at all, and the bot will set the kibana-ci commit check to success, which fixes issues we've had with docs changes for old branches.

@brianseeders brianseeders added Feature:CI Continuous integration release_note:skip Skip the PR/issue when compiling release notes v8.3.0 Team:Operations Kibana-Operations Team labels May 18, 2022
@brianseeders brianseeders marked this pull request as ready for review May 18, 2022 18:17
@brianseeders brianseeders requested a review from a team as a code owner May 18, 2022 18:17
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@brianseeders brianseeders enabled auto-merge (squash) May 18, 2022 18:28
@brianseeders brianseeders disabled auto-merge May 18, 2022 18:29
Copy link
Copy Markdown
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

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

I really with that JSON files could have comments, or that there was built in support for a better syntax... but either way these specific changes LGTM

@brianseeders
Copy link
Copy Markdown
Contributor Author

I really with that JSON files could have comments, or that there was built in support for a better syntax... but either way these specific changes LGTM

Yeah... I think I'm going to move it to json5 soon so we can have comments

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@brianseeders brianseeders merged commit dac92b2 into elastic:main May 18, 2022
@kibanamachine kibanamachine added the backport:skip This PR does not require backporting label May 18, 2022
@brianseeders brianseeders deleted the move-skip-config branch May 18, 2022 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:CI Continuous integration release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v8.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants