Skip to content

Fix restore cache condition#34

Merged
TravisCardwell merged 1 commit into
mainfrom
tcard/windows
Feb 20, 2026
Merged

Fix restore cache condition#34
TravisCardwell merged 1 commit into
mainfrom
tcard/windows

Conversation

@TravisCardwell

@TravisCardwell TravisCardwell commented Feb 20, 2026

Copy link
Copy Markdown
Collaborator

Manual Dispatch

Navigating to the "Actions" tab and clicking on the "Windows tests" workflow in the list on the left brings you to the Windows tests workflow page.

There is a "Run workflow" button awkwardly placed in between the header and the table body. Clicking on the button opens the options, and the green "Run workflow" button can be pressed to actually run the workflow.


For my test, I set the following options:

  • Branch: main
  • Versions: "9.6","9.12"
  • Enable cache: false
  • Debug with tmate after: checkout

The correct jobs are run, and both started the tmate sessions.

The "Restore cache" step ran, which is wrong. The condition is as follows:

${{ github.event_name != 'workflow_dispatch' || inputs.enable_cache }}

I bet it is a boolean vs. string issue. I am changing the condition to the following:

${{ github.event_name != 'workflow_dispatch' || inputs.enable_cache == 'true' }}

I am also changing the input configuration to quote "true" to emphasize that it is a string.

@TravisCardwell
TravisCardwell added this pull request to the merge queue Feb 20, 2026
Merged via the queue into main with commit 890271c Feb 20, 2026
11 checks passed
@TravisCardwell
TravisCardwell deleted the tcard/windows branch February 20, 2026 00:28
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