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

Add skip changelog label logic to pr-has-fragment #48

Merged
merged 6 commits into from
Jun 13, 2022

Conversation

endorama
Copy link
Member

@endorama endorama commented Jun 7, 2022

Closes #39

This PR adds logic to pr-has-fragment command to skip verification if the PR is labelled with a set of labels (currently hardcoded to skip-changelog and backport).

@endorama endorama requested a review from andresrc June 7, 2022 16:57
@elasticmachine
Copy link
Collaborator

elasticmachine commented Jun 7, 2022

💚 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 preview

Expand to view the summary

Build stats

  • Start Time: 2022-06-08T13:36:47.172+0000

  • Duration: 3 min 40 sec

Test stats 🧪

Test Results
Failed 0
Passed 55
Skipped 0
Total 55

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@endorama endorama mentioned this pull request Jun 7, 2022
With this change the pr-has-fragment command will check if the PR
has some specific labels and do not perform the fragment presence
check if yes.

The labels are currently hardcoded to skip-changelog and backport,
they will be made parametric in the future.
@endorama endorama requested review from lucian-ioan and removed request for andresrc June 9, 2022 15:16
Copy link
Collaborator

@lucian-ioan lucian-ioan left a comment

Choose a reason for hiding this comment

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

I've tested this and it works as intended. I think it's alright to leave the labels hardcoded for now.

@endorama endorama merged commit c1cbc70 into main Jun 13, 2022
@endorama endorama deleted the skip-changelog-labels branch June 13, 2022 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ignore PR without fragment and with specific label
3 participants