Skip to content

CI: Add cache_nonce env var, fix matrix.target.os key, do not cache ~/.cache/nim - #443

Open
moigagoo wants to merge 1 commit into
masterfrom
bugfix/ci_cache
Open

moigagoo wants to merge 1 commit into
masterfrom
bugfix/ci_cache

Conversation

@moigagoo

Copy link
Copy Markdown
Collaborator
  1. cache_nonce was not defined in ENV so the value would always be undefined.
  2. matrix.target.os replaced with runner.os so that the key and restore key match.
  3. caching ~/.cache/nim seems dangerous given that Nim 2.4 stores incremental compilation results there.

…/.cache/nim

1. cache_nonce was not defined in ENV so the value would always be
undefined.
2. matrix.target.os replaced with runner.os so that the key and restore
key match.
3. caching ~/.cache/nim seems dangerous given that Nim 2.4 stores
incremental compilation results there.
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