Skip to content

fix(ci): restrict PR checklist by author#680

Open
Molter73 wants to merge 1 commit into
mainfrom
mauro/fix/checklist-restrict-pr-author
Open

fix(ci): restrict PR checklist by author#680
Molter73 wants to merge 1 commit into
mainfrom
mauro/fix/checklist-restrict-pr-author

Conversation

@Molter73
Copy link
Copy Markdown
Collaborator

@Molter73 Molter73 commented May 18, 2026

Description

The existing check reacts to whoever performs an action, so pushing a commit to a PR opened by konflux bot will trigger the checklist validation. This is not a huge deal, but since MintMaker PRs have predefined descriptions, we should never attempt to validate them.

With this change we restrict the validation to PRs that are not opened by konflux bot.

For an example of a MintMaker PR running the checklist validation test after a push from another user see #679.

Checklist

  • Patch has a change log entry OR does not need one.
  • Investigated and inspected CI test results
  • Updated documentation accordingly

Automated testing

  • Added unit tests
  • Added integration tests
  • Added regression tests

If any of these don't apply, please comment below.

Testing Performed

This will need to be fully tested by merging and letting a MintMaker PR be opened.

@Molter73 Molter73 requested a review from a team as a code owner May 18, 2026 09:59
The existing check reacts to whoever performs an action, so pushing a
commit to a PR opened by the konflux bot will trigger the checklist
validation. This is not a huge deal, but since MintMaker PRs have
predefined descriptions, we should never attempt to validate them.

With this change we restrict the validation to PRs that are not opened
by konflux bot.

For an example of a MintMaker PR running the checklist validation test
after a push from another user see #679.
@Molter73 Molter73 force-pushed the mauro/fix/checklist-restrict-pr-author branch from 20f87c6 to fd74bc7 Compare May 18, 2026 10:13
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.

2 participants