Skip to content

Conversation

@sensei-hacker
Copy link
Member

@sensei-hacker sensei-hacker commented Nov 30, 2025

User description

Adds a GitHub Action that automatically comments on new Pull Requests targeting the master branch, suggesting that contributors consider targeting maintenance-9.x or maintenance-10.x branches instead when appropriate.

  • Triggers when a PR is opened targeting master
  • Adds an informational comment explaining when to use maintenance branches
  • Non-blocking - contributors can still merge to master if appropriate

PR Type

Enhancement


Description

  • Adds GitHub Action workflow for PR branch suggestions

  • Automatically comments on PRs targeting master branch

  • Suggests maintenance-9.x or maintenance-10.x branches

  • Provides guidance on branch selection criteria


Diagram Walkthrough

flowchart LR
  PR["PR opened on master"] -- "triggers workflow" --> Action["GitHub Action executes"]
  Action -- "posts comment" --> Comment["Branch suggestion comment"]
  Comment -- "guides contributor" --> Decision["Choose appropriate branch"]
Loading

File Walkthrough

Relevant files
Enhancement
pr-branch-suggestion.yml
New workflow for PR branch targeting suggestions                 

.github/workflows/pr-branch-suggestion.yml

  • Creates new GitHub Actions workflow triggered on PR open events
  • Targets PRs opened against the master branch
  • Uses github-script action to post automated comment
  • Comment explains when to use maintenance-9.x vs maintenance-10.x
    branches
  • Provides clear guidance on backward compatibility considerations
+37/-0   

@qodo-code-review
Copy link
Contributor

qodo-code-review bot commented Nov 30, 2025

PR Compliance Guide 🔍

All compliance sections have been disabled in the configurations.

Co-authored-by: qodo-merge-pro[bot] <151058649+qodo-merge-pro[bot]@users.noreply.github.com>
@sensei-hacker sensei-hacker merged commit 3e4efe1 into iNavFlight:master Nov 30, 2025
5 of 20 checks passed
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.

1 participant