Skip to content

gha: check-pr-branch: verify major version only#48177

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:relax_pr_check
Jul 19, 2024
Merged

gha: check-pr-branch: verify major version only#48177
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:relax_pr_check

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

We'll be using release branches for minor version updates, so instead of (e.g.) a 27.0 branch, we'll be using 27.x and continue using the branch for minor version updates.

This patch changes the validation step to only compare against the major version.

- A picture of a cute animal (not mandatory but encouraged)

@vvoland
Copy link
Copy Markdown
Contributor

vvoland commented Jul 18, 2024

image

The error message is a bit wrong now (it fails correctly though).

Comment thread .github/workflows/validate-pr.yml Outdated
@thaJeztah
Copy link
Copy Markdown
Member Author

@vvoland updated; PTAL (I also updated the backport)

Comment thread .github/workflows/validate-pr.yml Outdated
Comment thread .github/workflows/validate-pr.yml Outdated
Comment thread .github/workflows/validate-pr.yml Outdated
@thaJeztah
Copy link
Copy Markdown
Member Author

Ah, dang; YAML!

Error: /workflows/validate-pr.yml:61:1: [error] trailing spaces (trailing-spaces)
Error: /workflows/validate-pr.yml:64:1: [error] trailing spaces (trailing-spaces)

We'll be using release branches for minor version updates, so instead
of (e.g.) a 27.0 branch, we'll be using 27.x and continue using the
branch for minor version updates.

This patch changes the validation step to only compare against the
major version.

Co-authored-by: Cory Snider <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
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.

3 participants