Skip to content

Cache per runner.arch#430

Merged
castelao merged 1 commit into
mainfrom
fix/tox_cache_osx
May 21, 2026
Merged

Cache per runner.arch#430
castelao merged 1 commit into
mainfrom
fix/tox_cache_osx

Conversation

@castelao
Copy link
Copy Markdown
Member

Addressing the mix between intel & ARM.

Addressing the mix between intel & ARM.
@castelao castelao self-assigned this May 21, 2026
@castelao castelao requested a review from ppinchuk as a code owner May 21, 2026 04:12
Copilot AI review requested due to automatic review settings May 21, 2026 04:12
@castelao castelao added the bugfix Fixed a known bug label May 21, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the GitHub Actions tox cache keying strategy to avoid cache collisions between Intel and ARM runners, which can cause incorrect .tox/ reuse across architectures.

Changes:

  • Add ${{ runner.arch }} to the tox cache key and restore key prefix.
  • Bump the tox cache key version from v1 to v2 to ensure a clean cache namespace.

@castelao castelao merged commit cfaee7f into main May 21, 2026
20 checks passed
@castelao castelao deleted the fix/tox_cache_osx branch May 21, 2026 04:48
rajeee pushed a commit that referenced this pull request May 27, 2026
Addressing the mix between intel & ARM.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixed a known bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants