clawdie/colibri
1
0
Fork 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions 52

docs/pr-workflow-guide #269

Merged
clawdie merged 2 commits from docs/pr-workflow-guide into main 2026-06-28 10:57:52 +02:00
Conversation 0 Commits 2 Files changed 11 +284 -11
clawdie commented 2026-06-28 10:57:20 +02:00
Owner
Copy link
No description provided.
clawdie added 2 commits 2026-06-28 10:57:26 +02:00
rename golden tests to fixtures tests
Some checks are pending
CI / rust (pull_request) Waiting to run
Details
CI / markdown (pull_request) Waiting to run
Details
CI / port (pull_request) Waiting to run
Details
CI / agent-jail-pkgs (pull_request) Waiting to run
Details
0062cb13e3 
Replace jargon term 'golden tests' with clearer 'fixtures tests' throughout
the codebase. This aligns with the project's preference for explicit,
understandable terminology over obscure testing jargon.

Changes:
- Renamed crates/colibri-contracts/tests/golden.rs → fixtures.rs
- Updated comments in fixtures.rs and session.rs
- Updated docs: AGENTS.md, README.md, colibri.md, index.md
- Updated wiki: contracts.md, runtime-inventory.md, sl/contracts.md

Note: Kept intentional 'golden line' metaphor in COLIBRI-TOKENOMICS-TRIFECTA.md
as it refers to the concept, not the test file.

All gates pass:
- cargo fmt, clippy, test (14/14 in contracts)
- wiki-lint: 187/0
docs(wiki): add comprehensive PR workflow guide
Some checks are pending
CI / rust (pull_request) Waiting to run
Details
CI / markdown (pull_request) Waiting to run
Details
CI / port (pull_request) Waiting to run
Details
CI / agent-jail-pkgs (pull_request) Waiting to run
Details
4be669aa3f 
Document day-to-day PR workflow used throughout the project:
- What is a PR and why we use them
- Complete workflow from branch creation to merge
- Branch naming conventions (fix/, feat/, docs/, refactor/, test/)
- Commit message format with good/bad examples
- Review checklist and common patterns
- Merging strategy (squash vs regular)
- After merge cleanup steps
- Troubleshooting guide
- Real examples from recent PRs (eval-harness-phase1, smoke-to-test, sl-wiki-cross-links)

Captures the practical PR workflow we use daily in chat and makes
it accessible as reference documentation.

Sam & Claude
clawdie merged commit c6abca092d into main 2026-06-28 10:57:52 +02:00
clawdie deleted branch docs/pr-workflow-guide 2026-06-28 10:57:56 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
first-proof blocker

gates the first proven end-to-end

  
hardening

security/robustness follow-up, not a first-proof gate

No labels
first-proof blocker
hardening
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
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: clawdie/colibri#269
No description provided.
Delete branch "docs/pr-workflow-guide"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?