rasterstate/fj
rasterstate/fj
3
0
Fork 0
Code Issues 15 Pull requests Releases 5 Activity Actions

Add pr create fill flags #175

Closed
stephen wants to merge 1 commit from feat/165-pr-create-fill-codex into main
pull from: feat/165-pr-create-fill-codex
merge into: rasterstate:main
Conversation 1 Commits 1 Files changed 2 +272 -6
stephen commented 2026-06-15 15:27:03 +00:00
Owner
Copy link

Why

Support one-command PR creation from commit messages without new API calls.

Changes

  • Add --fill and --fill-first to fj pr create.
  • Resolve generated title/body through the create body path, with explicit title/body/-F taking precedence.
  • Document the gh mapping for fill flags.

Testing

  • cargo test -q create_parses_fill_flags
  • cargo test -q fill

Closes #165

Why Support one-command PR creation from commit messages without new API calls. Changes - Add --fill and --fill-first to fj pr create. - Resolve generated title/body through the create body path, with explicit title/body/-F taking precedence. - Document the gh mapping for fill flags. Testing - cargo test -q create_parses_fill_flags - cargo test -q fill Closes #165
Add PR create fill flags
All checks were successful
ci / check (pull_request) Successful in 9m51s
Details
ci / live-e2e (pull_request) Successful in 1m49s
Details
ci / coverage (pull_request) Successful in 1m49s
Details
7161e2642c
stephen closed this pull request 2026-06-15 15:36:16 +00:00
stephen commented 2026-06-15 15:36:16 +00:00
Author
Owner
Copy link

Closing as a duplicate of #174, which implements the same fj #165 pr create --fill/--fill-first feature with a dedicated pr_fill.rs module + git helpers (better factored for #172 to reuse). Thanks for the parallel take.

Closing as a duplicate of #174, which implements the same fj #165 `pr create --fill`/`--fill-first` feature with a dedicated pr_fill.rs module + git helpers (better factored for #172 to reuse). Thanks for the parallel take.
All checks were successful
ci / check (pull_request) Successful in 9m51s
Required
Details
ci / live-e2e (pull_request) Successful in 1m49s
Required
Details
ci / coverage (pull_request) Successful in 1m49s
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
backlog
Accepted but not scheduled

  
blocked
Blocked or stalled waiting on external progress

  
converted
Converted into tracked work

  
opportunity
Potential product or business opportunity

  
p1
Priority 1

  
p2
Priority 2

  
p3
Priority 3

  
parked
Paused intentionally until later

No labels
backlog
blocked
converted
opportunity
p1
p2
p3
parked
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
rasterstate/fj!175
No description provided.