alexion/dev
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs(skills): add nix let review rule #55

Merged
alexion merged 2 commits from docs/nix-review-rules into main 2026-09-16 14:20:00 -04:00
Conversation 0 Commits 2 Files Changed 6 +123 -131
alexion commented 2026-09-16 12:41:08 -04:00
Owner
Copy Link

Summary

  • Add a Nix style rule for minimizing shared convenience let bindings.
  • Tighten /review-nix output so findings cite reviewed code and the rule source.
  • Replace the partial Nix refactor tree with a complete entrypoint-ordered checklist.

Tests

  • Verified the Nix checklist matches all discovered Nix files under flake.nix, src/dotfiles/, and configs/nixos/.
## Summary - Add a Nix style rule for minimizing shared convenience `let` bindings. - Tighten `/review-nix` output so findings cite reviewed code and the rule source. - Replace the partial Nix refactor tree with a complete entrypoint-ordered checklist. ## Tests - Verified the Nix checklist matches all discovered Nix files under `flake.nix`, `src/dotfiles/`, and `configs/nixos/`.
alexion added 1 commit 2026-09-16 14:19:32 -04:00
refactor(dotfiles): use formatter output in fix app 82b727e437
alexion force-pushed docs/nix-review-rules from 66a1468187 to 82b727e437 2026-09-16 14:19:32 -04:00 Compare
alexion merged commit 82b727e437 into main 2026-09-16 14:20:00 -04:00
alexion deleted branch docs/nix-review-rules 2026-09-16 14:20:00 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
alexion
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: alexion/dev#55
Delete Branch "docs/nix-review-rules"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?