Skip to content

Add Woodpecker CI pipeline#3

Open
reef-ndb wants to merge 2 commits intomainfrom
add/woodpecker-ci
Open

Add Woodpecker CI pipeline#3
reef-ndb wants to merge 2 commits intomainfrom
add/woodpecker-ci

Conversation

@reef-ndb
Copy link
Collaborator

@reef-ndb reef-ndb commented Feb 5, 2026

Adds a .woodpecker.yml pipeline to run fmt, clippy, and tests on push/PR.

Summary by CodeRabbit

  • Chores

    • Added Woodpecker CI/CD pipeline configuration with automated code formatting, linting, and testing checks.
  • Refactor

    • Improved cross-stage job dependency resolution logic for better build coordination.

@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

Walkthrough

The changes introduce a Woodpecker CI pipeline configuration for a Rust project with steps for dependency installation, code formatting, linting, and testing. Additionally, job dependency matching logic in the orchestrator is simplified by removing stage-prefix gating, allowing direct linking of instances with matching job IDs across stages.

Changes

Cohort / File(s) Summary
CI Pipeline Configuration
.woodpecker.yml
New pipeline configuration defining triggers for push/pull_request on main branch with serial steps: install-deps (nodejs, npm, protobuf-compiler), fmt (rustfmt check), clippy (linting with warnings as errors), and test (cargo test). All steps use rust:1.85 image and depend on install-deps.
Orchestrator Job Matching
src/orchestrator/dag.rs
Simplifies dependent job matching by removing stage-prefix gating. Now directly links all concrete instances sharing the same job_id (including matrix-expanded variants) as dependencies, regardless of stage alignment. Reduces conditional logic while retaining error handling for unmatched dependencies.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 A pipeline takes shape with careful care,
Rust tests and clippy check everywhere!
Dependencies dance without stage-prefix fuss,
Matrix variants linked—no need for a muss!
Build automation hops along so fine,
Making sure the code stays clean and in line! 🏗️✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title 'Add Woodpecker CI pipeline' directly describes the main change in the changeset (.woodpecker.yml file), clearly summarizing the primary objective.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch add/woodpecker-ci

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant