Skip to content

Fix cache workflow environment context - #24

Merged
nicolaswill merged 1 commit into
mainfrom
nicolaswill-cache-aware-bundles
Jul 28, 2026
Merged

Fix cache workflow environment context#24
nicolaswill merged 1 commit into
mainfrom
nicolaswill-cache-aware-bundles

Conversation

@nicolaswill

Copy link
Copy Markdown
Contributor

The cache workflow failed before creating any jobs because the global env block referenced runner.temp, where the runner context is unavailable.

Use github.workspace for the persisted cache directory instead. The workflow now passes Actionlint validation.

@nicolaswill
nicolaswill requested a review from a team as a code owner July 28, 2026 00:02
Copilot AI review requested due to automatic review settings July 28, 2026 00:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes the codeql-compilation-caches GitHub Actions workflow failing at workflow-load time by removing the invalid workflow-level reference to the runner context and instead using a context that’s available in the global env block.

Changes:

  • Update CODEQL_BUNDLE_CACHE_DIR to use ${{ github.workspace }} instead of ${{ runner.temp }} at workflow scope.
  • Keep the cache directory stable within the workspace (.codeql-bundle-cache) so subsequent cache steps continue to reference a consistent location.
Show a summary per file
File Description
.github/workflows/codeql-compilation-caches.yml Fixes workflow-scope env context usage so the workflow can be parsed/validated and jobs can be created.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Low

@nicolaswill
nicolaswill merged commit 1d3eab8 into main Jul 28, 2026
6 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.

2 participants