Skip to content

CONTRIBUTING.md

Latest commit

 

History

History
76 lines (57 loc) · 2.78 KB

CONTRIBUTING.md

File metadata and controls

76 lines (57 loc) · 2.78 KB
Raw

Contributing

When contributing to this repository, please first discuss the change you wish to make via issue, email, or any other method with the owners of this repository, ensuring you follow the Code of Conduct.

Development Setup

Gitfleet is a Rust workspace. Build and test with Cargo:

cargo build                     # Compile the workspace.
cargo fmt --check               # Verify formatting.
cargo clippy -- -D warnings     # Run lints with warnings as errors.
cargo check                     # Type-check without codegen.
cargo test --workspace          # Run all unit and integration tests.
cargo llvm-cov --fail-under-lines 80  # Generate coverage report (80% gate).
cargo build --release           # Optimized release build.

Git hooks are managed by Lefthook. Install hooks once after cloning:

lefthook install

Pre-commit runs cargo fmt --check and cargo clippy -- -D warnings. Pre-push runs cargo test --workspace.

Architecture

Read AGENTS.md for the full architecture contract, crate boundaries, code style, testing strategy, and release rules. Key points:

  • gitfleet-core holds provider-neutral types, traits, and infrastructure.
  • gitfleet-providers is the only crate that calls reqwest.
  • gitfleet-cli is a thin surface over shared operations.
  • Provider wire types must be normalized before crossing the crate boundary.
  • Never add raw println! or eprintln! outside the output layer.
  • Use GitfleetError enum variants for expected failures.

Testing

Unit tests live inside source files in #[cfg(test)] mod tests {} blocks. Integration tests live in each crate's tests/ directory. Provider tests use wiremock for HTTP mocking and insta for snapshot normalization. Automated tests must never make real HTTP requests.

Live playbooks under gitfleet-playbooks/ validate command families against the real GitHub API. They require explicit credentials and clean up mutations during teardown.

Commit Convention

All commit messages must use a lowercase prefix followed by a colon and space:

  • feat: — new user-visible behavior
  • fix: — bug fix
  • refactor: — code restructure without behavior change
  • chore: — build, release, dependency, or metadata changes
  • tests: — test additions or modifications
  • ci: — CI/CD workflow changes
  • documentation: — documentation-only changes
  • repo: — project scaffolding

Subject line: imperative mood, no period, under 50 characters. No scopes. No body.

Pull Requests

  • Use the pull request template provided in the repository.
  • Ensure all gates pass before submitting.
  • Rebase your branch on main before opening a PR.
  • One logical change per PR.