Release/v1.0.0 rc1 - #118
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
WalkthroughThis PR adds deterministic release automation for CDT++, including metadata validation, offline changelog generation, annotated Git tag creation, release recipes, synchronized release documentation, and coverage-specific CTest timeout handling. ChangesRelease automation
Estimated code review effort: 4 (Complex) | ~60 minutes Sequence Diagram(s)sequenceDiagram
participant Developer
participant Justfile
participant generate_changelog
participant git_cliff
participant cdt_tag_release
participant Git
Developer->>Justfile: run changelog-unreleased
Justfile->>generate_changelog: pass release version
generate_changelog->>git_cliff: render changelog offline
git_cliff-->>generate_changelog: return release entries
generate_changelog-->>Developer: write dated CHANGELOG.md
Developer->>Justfile: run tag-check or tag
Justfile->>cdt_tag_release: pass release tag
cdt_tag_release->>Git: validate worktree and tag state
Git-->>cdt_tag_release: return preflight status
cdt_tag_release->>Git: create annotated tag
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/RELEASING.md`:
- Line 24: Remove the unused RELEASE_DATE assignment from the release
instructions, or update the documented workflow to actually consume it. Keep
CITATION.cff as the authoritative source for the release date and avoid implying
this variable affects release metadata.
In `@scripts/tests/test_generate_changelog.py`:
- Around line 29-30: Make the unittest exception assertions Ruff-compatible by
excluding PT027 for scripts/tests or adding targeted, justified suppressions at
scripts/tests/test_generate_changelog.py lines 29-30, 56-57, and 98-99, and
scripts/tests/test_tag_release.py lines 60-61, 102-103, and 112-113. Preserve
the existing unittest.TestCase assertions and ensure just python-lint passes.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 3449a280-c70e-4200-b385-9ea8a3bdab1d
📒 Files selected for processing (16)
.github/CONTRIBUTING.mdCHANGELOG.mdCITATION.cffJustfileREADME.mdcliff.tomldocs/RELEASING.mdpyproject.tomlscripts/generate_changelog.pyscripts/release_check.pyscripts/subprocess_utils.pyscripts/tag_release.pyscripts/tests/test_generate_changelog.pyscripts/tests/test_release_check.pyscripts/tests/test_tag_release.pytests/CMakeLists.txt
Summary by CodeRabbit
New Features
Documentation
Maintenance
1.0.0-rc1changelog and refreshed the released date for citation metadata.