Import archive-based cargo-binstall support#41
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Reviewer's GuideAdds a detailed ExecPlan document that defines how archive-based cargo-binstall support will be introduced for rendered app projects, covering constraints, risks, implementation milestones, and acceptance criteria without yet changing any template behaviour. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Summary
This branch carries the pre-implementation ExecPlan for importing archive-based
cargo binstallsupport into renderedappprojects. It plans the migration from the current direct-binarypkg-fmt = "bin"release contract to archive-based Linux GNU cargo-binstall artefacts staged throughleynos/shared-actions.Execplan: docs/execplans/cargo-binstall-support.md
Review walkthrough
appprojects.template/, and proved through rendered contract tests.Validation
make test 2>&1 | tee /tmp/test-agent-template-rust-cargo-binstall-support-plan.out:50 passed, 1 skipped in 55.93s.make test 2>&1 | tee /tmp/test-agent-template-rust-cargo-binstall-support-plan-amend.out:50 passed, 1 skipped in 47.42s.git diff --check: passed with no output.Notes
This is a pre-implementation plan pull request. It does not change generated template behaviour yet; it authorizes the follow-up implementation to add rendered contract tests, update app-only cargo-binstall metadata, introduce app-only release staging configuration, and move rendered app release staging onto the current pinned
leynos/shared-actionsrevision.The plan records that
leynos/shared-actionscurrently resolves to455d9ed03477c0026da96c2541ca26569a74acac, and the future implementation should use that pin unless a fresh audit deliberately updates the rendered workflow assertions in the same change.Summary by Sourcery
Documentation: