From 7e33538515fc2c25bd388d9590f4bbf7b0348cdc Mon Sep 17 00:00:00 2001 From: CepherXx <113156301+CepherXx@users.noreply.github.com> Date: Mon, 17 Aug 2026 02:44:42 -0700 Subject: [PATCH 1/3] docs: describe CI jobs accurately in CONTRIBUTING.md --- CONTRIBUTING.md | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 81e03f4..d48bffb 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -65,8 +65,13 @@ 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, both wasm builds, + clippy, and tests), `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/`, following the same layout as `demos/freelance-escrow`. @@ -213,7 +218,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, both wasm builds, clippy, tests), 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 From 7d0579c4dc45a3fb82d4b5931a71e52ed2a8d4d0 Mon Sep 17 00:00:00 2001 From: CepherXx <113156301+CepherXx@users.noreply.github.com> Date: Mon, 17 Aug 2026 17:48:35 -0700 Subject: [PATCH 2/3] fix: restore indentation and closing code fence --- CONTRIBUTING.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index d48bffb..cd9261a 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -65,14 +65,14 @@ tools/ scripts/ testnet-smoke.sh End-to-end check against real Stellar testnet infrastructure .github/workflows/ -ci.yml Runs three jobs on every push/PR: `test` (blocks + ci.yml Runs three jobs on every push/PR: `test` (blocks committed contract addresses, verifies workspace membership, fmt, shellcheck, both wasm builds, clippy, and tests), `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/`, following the same layout as `demos/freelance-escrow`. From 7ae31d4d4943eb8964a4af5a556c62ed3f470c6c Mon Sep 17 00:00:00 2001 From: CepherXx <113156301+CepherXx@users.noreply.github.com> Date: Tue, 18 Aug 2026 01:02:27 -0700 Subject: [PATCH 3/3] fix: correct wasm build step order and fence formatting --- CONTRIBUTING.md | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index cd9261a..fd928db 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -67,12 +67,14 @@ scripts/ .github/workflows/ ci.yml Runs three jobs on every push/PR: `test` (blocks committed contract addresses, verifies workspace - membership, fmt, shellcheck, both wasm builds, - clippy, and tests), `demo` (lint and build + 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/`, following the same layout as `demos/freelance-escrow`. @@ -218,7 +220,7 @@ changes in a single message; split them into separate commits instead. ## Opening a PR -CI must pass before merge: the `test` job (contract-address and workspace-membership checks, fmt, shellcheck, both wasm builds, clippy, tests), 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 +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