Skip to content

Remove unnecessary condition for merging codegen pull requests - #34

Merged
ptr727 merged 4 commits into
developfrom
prbranch
Apr 5, 2026
Merged

Remove unnecessary condition for merging codegen pull requests#34
ptr727 merged 4 commits into
developfrom
prbranch

Conversation

@ptr727

@ptr727 ptr727 commented Apr 5, 2026

Copy link
Copy Markdown
Owner

Signed-off-by: Pieter Viljoen ptr727@users.noreply.github.com

Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 5, 2026 18:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts the GitHub Actions workflow that auto-enables merging for codegen pull requests by loosening the job-level condition used to identify eligible PRs.

Changes:

  • Removes the github.actor == 'github-actions[bot]' requirement from the merge-codegen job condition.
  • Keeps gating on the codegen branch name and requiring the PR to originate from the same repository.

Comment thread .github/workflows/merge-bot-pull-request.yml Outdated
ptr727 added 2 commits April 5, 2026 11:18
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comment thread README.md
Comment thread .github/copilot-instructions.md Outdated
…uest workflow

Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
@ptr727
ptr727 merged commit f16acb7 into develop Apr 5, 2026
19 checks passed
@ptr727
ptr727 deleted the prbranch branch April 5, 2026 21: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