Personal/my-nix-lib
Watch
2
0
Fork
You've already forked my-nix-lib
 0
Code Issues 1 Pull requests 1 Projects Releases 2 Packages Wiki Activity Actions

ci(verify-nix): Init verify pipeline #9

Merged
martin merged 3 commits from verify-pipeline into main 2026-08-30 14:55:22 +00:00
Conversation 1 Commits 3 Files changed 4 +54 -10
martin commented 2026-08-24 19:07:46 +00:00
Owner
Copy link
No description provided.
martin self-assigned this 2026-08-24 19:07:46 +00:00
ci(verify-nix): Init verify pipeline
Some checks failed
ci/woodpecker/pr/pr-review Pipeline was successful
Details
ci/woodpecker/pr/verify-nix Pipeline failed
Details
3c03e558df
clank-bot commented 2026-08-24 19:08:27 +00:00
Member
Copy link

Review Summary

The PR adds a Woodpecker CI pipeline (.woodpecker/verify-nix.yaml) for verifying Nix flake changes, plus a minor justfile fix.

Changes:

  • New .woodpecker/verify-nix.yaml: Well-structured pipeline triggering on manual runs, PRs affecting Nix files/tests, and pushes to default branch. Uses Cachix for caching, runs nix flake check --print-build-logs and unit tests via nix-unit. Proper path filtering avoids unnecessary runs.
  • justfile: Fixes template variable syntax from {{version}} to {{ version }} (adding space).

Assessment:

  • Correctness: Pipeline configuration is correct and follows Nix/CI best practices
  • Security: CACHIX_AUTH_TOKEN properly sourced from secret
  • Performance: Cachix caching enabled for faster builds
  • Code quality: Clean, maintainable configuration
  • Best practices: Comprehensive verification (flake check + unit tests), build logs for debugging, appropriate NixOS 26.05 image

No issues found.

*🤖 Review by opencode (opencode/nemotron-3-ultra-free) — verdict: approve

# Review Summary The PR adds a Woodpecker CI pipeline (`.woodpecker/verify-nix.yaml`) for verifying Nix flake changes, plus a minor justfile fix. **Changes:** - New `.woodpecker/verify-nix.yaml`: Well-structured pipeline triggering on manual runs, PRs affecting Nix files/tests, and pushes to default branch. Uses Cachix for caching, runs `nix flake check --print-build-logs` and unit tests via `nix-unit`. Proper path filtering avoids unnecessary runs. - `justfile`: Fixes template variable syntax from `{{version}}` to `{{ version }}` (adding space). **Assessment:** - Correctness: Pipeline configuration is correct and follows Nix/CI best practices - Security: CACHIX_AUTH_TOKEN properly sourced from secret - Performance: Cachix caching enabled for faster builds - Code quality: Clean, maintainable configuration - Best practices: Comprehensive verification (flake check + unit tests), build logs for debugging, appropriate NixOS 26.05 image No issues found. --- *🤖 Review by opencode (opencode/nemotron-3-ultra-free) — verdict: **approve** <!-- forgejo-opencode-review -->
chore: Format
Some checks failed
ci/woodpecker/pr/pr-review Pipeline was successful
Details
ci/woodpecker/pr/verify-nix Pipeline failed
Details
70f7e64199
Nix unit uses input from flake inputs
Some checks failed
ci/woodpecker/pr/pr-review Pipeline failed
Details
ci/woodpecker/pr/verify-nix Pipeline was canceled
Details
52ac369124
martin force-pushed verify-pipeline from 52ac369124
Some checks failed
ci/woodpecker/pr/pr-review Pipeline failed
Details
ci/woodpecker/pr/verify-nix Pipeline was canceled
Details
to 84f9a0e778
Some checks failed
ci/woodpecker/pr/pr-review Pipeline failed
Details
ci/woodpecker/pr/verify-nix Pipeline was successful
Details
2026-08-30 14:52:30 +00:00 Compare
martin merged commit 35794e1a60 into main 2026-08-30 14:55:22 +00:00
martin deleted branch verify-pipeline 2026-08-30 14:55:22 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
clank-bot
Labels
Clear labels
  
automerge
Merge pull requests automatically after checks have completed

  
bug
Something is not working

  
dependencies
Dependency updates

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
migrations
Automated repo migration

  
question
More information is needed

  
vulnerability
Vulnerability update

  
wontfix
This won't be fixed

No labels
automerge
bug
dependencies
duplicate
enhancement
help wanted
invalid
migrations
question
vulnerability
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
martin
2 participants
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
Personal/my-nix-lib!9
No description provided.