Skip to content

fix(common): clarify crate-level doc wording - #3370

Merged
serrrfirat merged 1 commit into
mainfrom
fix/common-doc-wording
May 7, 2026
Merged

serrrfirat merged 1 commit into
mainfrom
fix/common-doc-wording

Conversation

@nickpismenkov

Copy link
Copy Markdown
Contributor

Summary

Tiny wording tweak in the ironclaw_common crate-level doc comment: for the IronClaw workspace → used across the IronClaw workspace.

Why

This is a deliberate one-word change to give release-plz a leaf-crate source change to detect. Since ironclaw-v0.27.0 (Apr 29), zero commits have touched crates/ironclaw_common/, crates/ironclaw_safety/, or crates/ironclaw_skills/ — and release-plz only proposes a release PR when at least one leaf crate has source-path changes since its last tag (the root ironclaw package is suppressed by the local > registry guard at release_plz_core/src/command/update/updater.rs:688).

Every prior release window happened to have leaf-crate churn (v0.24→25: 6/6/6 commits, v0.25→26: 14/0/6, v0.26→27: 6/2/3). v0.27→HEAD is the first window with 0/0/0 — all ~100 commits since v0.27.0 have been in src/ or workspace plumbing.

After this merges, release-plz will detect the leaf change, propose a patch bump for ironclaw_common, and cascade the root ironclaw package to 0.28.0. From there the standard release flow takes over (review the bot's chore PR → merge → tag → cargo-dist).

A more durable fix (e.g. release_always = true on the ironclaw [[package]] in release-plz.toml) is a separate conversation; this PR only unblocks the v0.28.0 cut.

Test plan

  • CI green on this PR
  • After merge, a chore: release PR appears from app/ironclaw-ci proposing ironclaw 0.27.0 → 0.28.0

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions Bot added size: XS < 10 changed lines (excluding docs) risk: low Changes to docs, tests, or low-risk modules contributor: core 20+ merged PRs labels May 7, 2026
@nickpismenkov nickpismenkov added the skip-regression-check Bypass regression test CI gate (tests exist but not in tests/ dir) label May 7, 2026
@nickpismenkov nickpismenkov reopened this May 7, 2026
@serrrfirat

Copy link
Copy Markdown
Collaborator

lol

@serrrfirat
serrrfirat merged commit 487d181 into main May 7, 2026
55 of 69 checks passed
@serrrfirat
serrrfirat deleted the fix/common-doc-wording branch May 7, 2026 18:31
@ironclaw-ci ironclaw-ci Bot mentioned this pull request May 7, 2026
This was referenced May 7, 2026
theredspoon pushed a commit to theredspoon/ironclaw that referenced this pull request Jun 21, 2026
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor: core 20+ merged PRs risk: low Changes to docs, tests, or low-risk modules size: XS < 10 changed lines (excluding docs) skip-regression-check Bypass regression test CI gate (tests exist but not in tests/ dir)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants