Skip to content

Update .backportrc.json: remove 7.x preset#5501

Merged
joepeeples merged 2 commits into
mainfrom
backport-tool-fix
Jul 11, 2024
Merged

Update .backportrc.json: remove 7.x preset#5501
joepeeples merged 2 commits into
mainfrom
backport-tool-fix

Conversation

@joepeeples

@joepeeples joepeeples commented Jul 3, 2024

Copy link
Copy Markdown
Contributor

Removes syntax that creates a pre-selected (or "checked")7.x option every time you run the backport tool. We used to do this during the 7.x series, but we no longer maintain such a branch and don't need it as a pre-selected option.

Backport tool docs: https://github.com/sorenlouv/backport/blob/main/docs/config-file-options.md#targetbranchchoices-required
(The documented setting names are slightly different from what's in our config file. Maybe they were renamed at some point but maintain backwards compatibility with our names?)

@joepeeples
joepeeples marked this pull request as ready for review July 3, 2024 15:37
@joepeeples
joepeeples requested a review from a team as a code owner July 3, 2024 15:37
@github-actions

github-actions Bot commented Jul 3, 2024

Copy link
Copy Markdown

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@joepeeples
joepeeples requested a review from jmikell821 July 3, 2024 15:43
@joepeeples

Copy link
Copy Markdown
Contributor Author

FYI: I don't think we need to backport this change to previous branches, since we usually run the backport tool from main, not a versioned branch.

@joepeeples
joepeeples merged commit 43b22d3 into main Jul 11, 2024
@joepeeples
joepeeples deleted the backport-tool-fix branch September 12, 2024 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants