Skip to content

argos: keep PR descriptions about the change, not a file list #28

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bracyw merged 1 commit into from
Oct 8, 2026
Merged

argos: keep PR descriptions about the change, not a file list #28

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ...application-software/argos/workspaces/argos-dev/.claude/skills/open-pr/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,4 @@ description: Run pre-PR checks, push the branch, and open a draft pull request
---
From the branch's worktree, run `bash <workspace>/.claude/skills/open-pr/scripts/pr-checks.sh` (clean tree, commit format, frontend lint if changed, conflicts with `origin/develop`); stop and fix on any FAIL. Then push and run `gh pr create --draft --base develop --head <branch> --title "<concise imperative title>" --body-file /tmp/<branch>-pr-body.md --assignee @me --label "by: <ai|ai-assisted|human>" --label <area>...` (written-by plus area labels: `frontend`, `backend`, `devops`), and report the URL.

**PR body:** fill `.github/pull_request_template.md` from the diff. Follow each section's comment: Changes in about 50 words, then delete the comments and any optional or non-applicable sections. Check off the Checklist and end with `Closes #{ticket}`. For UI changes put `_screenshot pending_` and remind the user to drag-drop screenshots from `pictures/<branch>/`. Write it to `/tmp/<branch>-pr-body.md`.
**PR body:** fill `.github/pull_request_template.md` from the diff. Follow each section's comment: Changes in about 50 words. Describe the change, not the file list: don't enumerate components, classes or files the diff already shows. Name a specific one only when its behaviour changes in a way a reviewer must know. Then delete the comments and any optional or non-applicable sections. Check off the Checklist and end with `Closes #{ticket}`. For UI changes put `_screenshot pending_` and remind the user to drag-drop screenshots from `pictures/<branch>/`. Write it to `/tmp/<branch>-pr-body.md`.
2 changes: 1 addition & 1 deletion ...plication-software/argos/workspaces/argos-dev/.claude/skills/update-pr/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,4 @@ description: Update the current branch's PR description to reflect the latest ch
---
Rewrite the current branch's PR body (`gh pr view`) to match `git diff develop...HEAD`. Keep human-written text that's still accurate, `user-attachments` screenshots, and `Closes`/`Fixes` refs. Apply it with `gh pr edit --body-file`, and add any missing written-by or area labels with `--add-label`.

**PR body:** fill `.github/pull_request_template.md` from the diff. Follow each section's comment: Changes in about 50 words, then delete the comments and any optional or non-applicable sections. Check off the Checklist and end with `Closes #{ticket}`. For UI changes put `_screenshot pending_` and remind the user to drag-drop screenshots from `pictures/<branch>/`. Write it to `/tmp/<branch>-pr-body.md`.
**PR body:** fill `.github/pull_request_template.md` from the diff. Follow each section's comment: Changes in about 50 words. Describe the change, not the file list: don't enumerate components, classes or files the diff already shows. Name a specific one only when its behaviour changes in a way a reviewer must know. Then delete the comments and any optional or non-applicable sections. Check off the Checklist and end with `Closes #{ticket}`. For UI changes put `_screenshot pending_` and remind the user to drag-drop screenshots from `pictures/<branch>/`. Write it to `/tmp/<branch>-pr-body.md`.
Loading