Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use GITHUB_TOKEN for pages deployment #346

Merged
merged 1 commit into from
May 20, 2021

Conversation

milesfrain
Copy link
Member

@milesfrain milesfrain commented May 20, 2021

Suggested in #345

@milesfrain
Copy link
Member Author

Don't know why we're getting a cache miss.

An earlier job creates the cache:

Cache not found for input keys: Linux-spago-969dc6fff69f8364c7c580e2da1a2c51397b149f53bacedb551a4d08ac5602f4
Cache saved with key: Linux-spago-969dc6fff69f8364c7c580e2da1a2c51397b149f53bacedb551a4d08ac5602f4

https://github.com/purescript-contrib/purescript-book/runs/2614884263?check_suite_focus=true

Then the job for this PR unexpectedly can't find that cache with the same key:

Cache not found for input keys: Linux-spago-969dc6fff69f8364c7c580e2da1a2c51397b149f53bacedb551a4d08ac5602f4

https://github.com/purescript-contrib/purescript-book/runs/2632204894?check_suite_focus=true

@milesfrain milesfrain merged commit e7ab775 into purescript-contrib:master May 20, 2021
@milesfrain milesfrain deleted the pages-token branch May 20, 2021 16:58
@milesfrain
Copy link
Member Author

Don't know why we're getting a cache miss.

Looks like this is due to security settings where caches are only available to branch descendants. One fix is to create these caches on the default / master branch too.
https://docs.github.com/en/actions/guides/caching-dependencies-to-speed-up-workflows#restrictions-for-accessing-a-cache

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant