Skip to content

Make release CLI packaging idempotent - #23

Merged
shps951023 merged 1 commit into
mainfrom
fix/release-cli-staging
Sep 15, 2026
Merged

shps951023 merged 1 commit into
mainfrom
fix/release-cli-staging

Conversation

@shps951023

Copy link
Copy Markdown
Member

Summary

  • stage the Windows CLI archive under RUNNER_TEMP instead of cached target/release-cli
  • clear the staging directory before packaging
  • allow the destination ZIP to be safely overwritten during release repair runs

The v0.5.0 repair run passed verification and Browser Lab packaging, then failed because Swatinem/rust-cache restored target/release-cli and New-Item rejected the existing directory: https://github.com/mini-software/MiniExcel-Rust/actions/runs/34975147809

After this merges, rerun the Release workflow with workflow_dispatch and tag=v0.5.0.

Validation

  • ran the updated packaging commands twice consecutively
  • verified the ZIP contains LICENSE, README.md, and miniexcel.exe
  • actionlint .github/workflows/release.yml
  • VS Code workflow diagnostics: no errors
  • git diff --check

@coderabbitai

coderabbitai Bot commented Sep 15, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 21 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

This review ran on the open-source allowance, not this organization's plan, because the pull request author doesn't have an assigned seat. Waiting won't change this — ask an organization admin to assign them a seat, or add seats in Billing if every seat is already assigned, then retry.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: b2947383-c511-46e3-8ea2-313812fda391

📥 Commits

Reviewing files that changed from the base of the PR and between 8e3392e and 4d6b276.

📒 Files selected for processing (1)
  • .github/workflows/release.yml

Comment @coderabbitai help to get the list of available commands.

@shps951023
shps951023 merged commit fa386bb into main Sep 15, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant