Add PR message flags for non-interactive use #12184
Merged
+157
−33
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds support for non-interactive PR creation by introducing --message (-m) and --file (-F) flags to the but pr new command. These flags allow users to specify PR title and description without requiring editor prompts, making the command suitable for CI/CD and automation scenarios.
Changes:
- Added
--messageand--fileflags tobut pr newcommand for non-interactive PR message specification - Implemented message parsing logic that treats the first line as title and subsequent lines as description
- Added validation to detect non-interactive environments and require appropriate flags
- Limited custom PR messages to the selected branch only (dependent branches in stacks use default messages)
- Updated skill reference documentation to reflect new flags and behavior
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| crates/but/src/args/forge.rs | Added message and file arguments with mutual exclusivity constraints for the New subcommand |
| crates/but/src/lib.rs | Implemented file reading, message parsing, non-interactive environment detection, and updated create_pr call signature |
| crates/but/src/command/legacy/forge/review.rs | Added PrMessage struct, parse_pr_message function, updated function signatures to thread message through the call chain, limited custom messages to selected branches, and added comprehensive unit tests |
| crates/but/skill/references/reference.md | Updated documentation with examples of -F and -m flags, non-interactive usage guidance, and stacked branch behavior |
| crates/but/skill/SKILL.md | Added quick reference for -F flag and noted stacked branch behavior |
krlvi
force-pushed
the
pr-message-noninteractive
branch
2 times, most recently
from
83411d4 to
6fa7c82
Compare
krlvi
force-pushed
the
pr-message-noninteractive
branch
from
6fa7c82 to
2313a79
Compare
krlvi
force-pushed
the
pr-message-noninteractive
branch
from
2313a79 to
0cf2848
Compare
Update PR message docs in skill references Limit custom PR message to selected branch
krlvi
force-pushed
the
pr-message-noninteractive
branch
from
0cf2848 to
7811643
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update PR message docs in skill references
Limit custom PR message to selected branch