mirror of
https://github.com/srid/nixos-config.git
synced 2026-04-11 05:45:27 +08:00
pr
This commit is contained in:
parent
c544a84050
commit
f7ecd7ba63
4 changed files with 16 additions and 4 deletions
|
|
@ -35,7 +35,7 @@ Take a GitHub issue, prompt, or task description and execute it top-to-bottom: i
|
|||
### 3. **Open Draft PR**
|
||||
|
||||
- Push the branch and open a **draft** pull request using `gh pr create --draft`.
|
||||
- **Use the `github-pr` skill** for the title and body — no bullet-list dumps.
|
||||
- **MANDATORY**: Load the `github-pr` skill (via `Skill` tool) BEFORE writing the PR title/body. Follow it exactly — paragraph-based descriptions, no bullet-list dumps.
|
||||
|
||||
### 4. **CI Loop**
|
||||
|
||||
|
|
|
|||
|
|
@ -14,3 +14,4 @@
|
|||
|
||||
- **nix tools**: If any tool is unavailable, get it from nixpkgs: `nix run nixpkgs#<tool>`.
|
||||
- **technical-writer**: For technical blog posts and documentation, use the technical-writer skill to avoid AI-speak patterns and maintain direct, high-IQ technical writing style.
|
||||
- **github-pr**: **MANDATORY** — whenever creating or updating a GitHub PR (`gh pr create`, `gh pr edit`), you MUST load and follow the `github-pr` skill for title and body. No exceptions. No bullet-list dumps. Read the skill first, then write the PR.
|
||||
|
|
|
|||
|
|
@ -38,6 +38,17 @@ Write PR descriptions that fellow devs actually want to read.
|
|||
- Link to issues/discussions where relevant (`Closes #123`, `See #456`)
|
||||
- If the PR is trivial (typo fix, version bump), a one-liner body is fine
|
||||
|
||||
## Try it locally
|
||||
|
||||
If the repo is a Nix flake and the PR branch contains a buildable output (package, NixOS config, etc.), include a "Try it locally" section at the end of the body. Use the GitHub owner/repo and branch name to construct the command:
|
||||
|
||||
```
|
||||
### Try it locally
|
||||
`nix run github:<owner>/<repo>/<branch>`
|
||||
```
|
||||
|
||||
Adjust the command as needed — `nix build` for non-runnable outputs, add `#<output>` if the default package isn't the relevant one. Omit this section entirely if the change isn't meaningfully testable via `nix run/build` (e.g., CI-only changes, documentation, non-Nix repos).
|
||||
|
||||
## Updating existing PRs
|
||||
|
||||
When the user pushes further changes to an already-PR'd branch:
|
||||
|
|
|
|||
6
flake.lock
generated
6
flake.lock
generated
|
|
@ -862,11 +862,11 @@
|
|||
"systems": "systems_3"
|
||||
},
|
||||
"locked": {
|
||||
"lastModified": 1774230611,
|
||||
"narHash": "sha256-cOiYBVZIGzCE6r9NM4F8vyMptKahKL3+LNb1RIoCj3U=",
|
||||
"lastModified": 1774271165,
|
||||
"narHash": "sha256-55d2JYsay/SnAZcA2eOT6H/miLa685+A/A2lYf9MxFI=",
|
||||
"owner": "juspay",
|
||||
"repo": "kolu",
|
||||
"rev": "86689beb1498f0ba432d059bf86f58adb97d5aaf",
|
||||
"rev": "f30acd88f63c4edb77b17c665bb8a5a8d4a91bdd",
|
||||
"type": "github"
|
||||
},
|
||||
"original": {
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue