diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 81e03f4..fd928db 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -65,7 +65,14 @@ tools/ scripts/ testnet-smoke.sh End-to-end check against real Stellar testnet infrastructure .github/workflows/ - ci.yml Runs fmt, clippy, tests, and the wasm build on every push/PR + ci.yml Runs three jobs on every push/PR: `test` (blocks + committed contract addresses, verifies workspace + membership, fmt, shellcheck, builds tholos's + wasm, clippy, tests, then a second workspace-wide + lib wasm build), `demo` (lint and build + demos/freelance-escrow), and `sdk` (checks + packages/tholos-sdk's generated bindings for + drift, then builds it) ``` Additional demo apps should each live as their own directory under `demos/`, @@ -213,7 +220,7 @@ changes in a single message; split them into separate commits instead. ## Opening a PR -CI (fmt, clippy, tests, wasm build) must pass before merge. The PR template +CI must pass before merge: the `test` job (contract-address and workspace-membership checks, fmt, shellcheck, builds tholos's wasm, clippy, tests, then a second workspace-wide lib wasm build), the `demo` job (lint and build demos/freelance-escrow), and the `sdk` job (bindings-drift check and build for packages/tholos-sdk). The PR template (`.github/pull_request_template.md`) is pre-filled when you open a PR; fill it out rather than deleting it. If the change affects bond amounts, resolver behavior, or anything with an economic consequence, say so explicitly in the summary so it's easy