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

ci: fix branch creation and mergify file #4391

Merged
merged 3 commits into from
Oct 19, 2024

Conversation

v1v
Copy link
Member

@v1v v1v commented Oct 17, 2024

I fixed it partially, but missed that I need to set the branches correctly after. moving to 8.x

Copy link
Contributor

mergify bot commented Oct 17, 2024

This pull request does not have a backport label. Could you fix it @v1v? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-/d./d is the label to automatically backport to the /d./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip notification from the automated backport with mergify label Oct 17, 2024
@v1v v1v marked this pull request as ready for review October 17, 2024 17:21
@v1v v1v requested a review from a team as a code owner October 17, 2024 17:21
@v1v v1v self-assigned this Oct 17, 2024
echo ' - "backport"' >> ../.mergify.yml
echo ' branches:' >> ../.mergify.yml
echo ' - "$(PROJECT_MAJOR_VERSION).$(PROJECT_MINOR_VERSION)"' >> ../.mergify.yml
echo ' conditions:' >> ../.mergify.yml
Copy link
Member Author

Choose a reason for hiding this comment

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

only a cosmetic change and the fix in L44

echo ' title: "[{{ destination_branch }}](backport #{{ number }}) {{ title }}"' >> ../.mergify.yml

## @help:prepare-major-minor-release:Prepare a major/minor release by creating a new branch reference.
.PHONY: prepare-major-minor-release
prepare-major-minor-release:
git checkout 8.x
Copy link
Member Author

Choose a reason for hiding this comment

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

we release from 8.x now, so it's required to checkout 8.x so we can create the branch from

@@ -68,6 +69,7 @@ create-branch-major-minor-release:
## @help:prepare-next-release:Prepare the original branch for the next release cycle.
.PHONY: prepare-next-release
prepare-next-release:
git checkout main
Copy link
Member Author

Choose a reason for hiding this comment

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

need main, mergify uses main only

.github/Makefile Outdated Show resolved Hide resolved
@v1v v1v merged commit 5229080 into main Oct 19, 2024
3 checks passed
@v1v v1v deleted the feature/fix-automation-second-attempt branch October 19, 2024 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-skip Skip notification from the automated backport with mergify needs-writer-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants