Skip to content

fix: simplify artifact cache pruning#194

Closed
peters wants to merge 2 commits into
mainfrom
fix/artifact-cache-pruning
Closed

fix: simplify artifact cache pruning#194
peters wants to merge 2 commits into
mainfrom
fix/artifact-cache-pruning

Conversation

@peters

@peters peters commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the extra install artifact cache prune-stat wrapper and use the shared prune helper directly
  • keep CLI prune logging concise while making GUI/headless install pruning silent best-effort
  • keep the release workflow on NuGet.org publishing and the supported checkout action version

Validation

  • ./scripts/sync-surge-core-vendor.sh --check
  • ./scripts/check-version-sync.sh
  • cargo fmt --all -- --check
  • RUSTFLAGS="-D warnings" cargo test --workspace
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo clippy --workspace --lib --bins --examples -- -D warnings -D clippy::unwrap_used -D clippy::expect_used
  • cargo clippy --workspace --all-targets --all-features -- -D warnings -W clippy::pedantic
  • dotnet format dotnet/Surge.slnx --verify-no-changes
  • dotnet test dotnet/Surge.slnx --configuration Release

Notes

  • The first local cargo test attempt needed network access to download the updated sysinfo crate from the refreshed base.
  • The first local dotnet format attempt failed during restore under the sandbox, then passed with network-enabled restore.

@peters peters closed this Jul 4, 2026
@peters peters deleted the fix/artifact-cache-pruning branch July 4, 2026 07:20
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