ci: replace direct-push release with a PR-based release flow - #82
Conversation
Identical rollout of the node-datto-rmm canary (task_1788457898992), including the eslint-globals and App-token contents:read fixes found during rollout. Fixes GH006.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 10 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe release workflow now detects incomplete release artifacts, prepares release files through a dry-run script, publishes missing artifacts, and creates or updates a release pull request targeting ChangesRelease flow
Estimated code review effort: 4 (Complex) | ~45 minutes Merge Risk: ⚪ Minimal · up to This change moves release preparation to a pull request flow while retaining resumable publication of missing artifacts. No concrete current-head merge-blocking risk remains. Sequence Diagram(s)sequenceDiagram
participant Main as main push
participant Workflow as release workflow
participant Prepare as prepare-release.mjs
participant GitHub as GitHub
participant Npm as npm registry
Main->>Workflow: start serialized release run
Workflow->>GitHub: check tag and release
Workflow->>Npm: check package publication
Workflow->>Prepare: calculate and prepare release files
Prepare-->>Workflow: version and release_needed output
Workflow->>GitHub: update release/next and release pull request
Workflow->>Npm: publish missing package
Workflow->>GitHub: create missing tag and release
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Rollout of the node-datto-rmm canary (fully verified live, including all fixes found during rollout). Identical diff. Fixes GH006. task_1788457898992.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Summary by CodeRabbit