Skip to content

[pull] master from ktmud:master - #9

Merged
pull[bot] merged 6 commits into
apache-superset:masterfrom
ktmud:master
Sep 13, 2026
Merged

pull[bot] merged 6 commits into
apache-superset:masterfrom
ktmud:master

Conversation

@pull

@pull pull Bot commented Sep 13, 2026 •

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

ktmud and others added 6 commits September 12, 2026 18:54
GitHub shut down the legacy cache service that the vendored `actions/cache`
sources (pinned as a git dependency) talked to, so `cache-restore` and
`cache-save` could no longer work. This replaces that dependency with the
`@actions/cache` toolkit package and calls it directly, which also removes
the monkey patching of `@actions/core` states.

- Run the action on the `node20` runtime.
- `cache-restore`/`cache-save` use the same node binary that runs the action.
  Fixes #69, supersedes #70.
- `cache-save` works without a prior `cache-restore` in the same step.
- Cache states are persisted under `RUNNER_TEMP`, keyed by run id.
- Modernize the dev toolchain and regenerate the stale lockfile.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T1TDGEfhhbnNCyvHjMLUm8
On Windows, the action sourced the bundled bashlib with a backslash path
that bash treated as escape characters, so every shortcut command failed
with `cache-save: command not found`.

- Convert paths to forward slashes and quote them when sourcing bashlib
  files and when passing the node binary to bash.
- Use `os.homedir()` and `%LOCALAPPDATA%` for the default `pip`/`npm`
  cache paths on Windows.
- Make `npm run clean` cross-platform, force LF line endings, add
  `windows-latest` to the CI matrix.

Fixes #74

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T1TDGEfhhbnNCyvHjMLUm8
The list of cache paths is mixed into the cache key hash. Since those paths
usually derive from `GITHUB_WORKSPACE` or `HOME`, which differ between hosted
and self-hosted runners, the same config produced different keys on different
runners. Replace runner specific directories with placeholders before hashing
and document how keys are computed.

Fixes #57

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T1TDGEfhhbnNCyvHjMLUm8
`cache-check <name>` looks up a cache without downloading it. It exits with
0 when a cache with the exact primary key exists, 2 when only a restore key
matched and 1 when nothing was found. Also document how to invalidate or
re-upload a bad cache.

Fixes #34
Fixes #55

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T1TDGEfhhbnNCyvHjMLUm8
- @actions/cache 6.2, @actions/core 3.0, @actions/exec 3.0 and
  @actions/glob 0.7 are ESM only, so the package is now `"type": "module"`
  and ncc emits ESM bundles.
- Run on the `node24` runtime.
- Drop `hasha` in favor of node's built-in `crypto`.
- Replace Jest with vitest 5; ESLint 10 with a flat config; ncc 0.45.
- TypeScript stays on 5.9 (the 7.x native port is not supported by
  typescript-eslint or ncc's loader yet).
- Replace the renovate config with a dependabot config.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T1TDGEfhhbnNCyvHjMLUm8
- README: point examples at `@v2`, add a "Migrating from v1" section.
- Bump package version to 2.0.0.
- CI: actions/checkout v7, actions/setup-node v7, codecov-action v7.
- eslint 10.10.0.
- Dependabot: ignore major bumps of `@types/node` and `typescript`.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T1TDGEfhhbnNCyvHjMLUm8
@pull pull Bot locked and limited conversation to collaborators Sep 13, 2026
@pull pull Bot added the ⤵️ pull label Sep 13, 2026
@pull
pull Bot merged commit 8d45e2e into apache-superset:master Sep 13, 2026
1 check was pending
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant