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

Allow commits with /nocache to not use the cache #2909

Merged
merged 3 commits into from Dec 10, 2021

Conversation

tomaarsen
Copy link
Member

@tomaarsen tomaarsen commented Dec 10, 2021

Hello!

Pull request overview

  • If a commit message contains /nocache, then it will not use the existing cache.

Details

From now on, a commit message containing /nocache will cause a CI run without the existing cache. Note that this does not refresh the cache, it just ignores the cache for that commit only. On the develop branch, it is recommended (for admins) to go to Secrets > CACHE_VERSION and change the value, after which you can go to Actions and "Restart this workflow". No new commit required here.

However, this does not work on PRs. That's what this PR is for. If any PR in the future is failing because of cache issues (which would only be if nltk_data was updated very recently), then an empty commit can be pushed with git commit --allow-empty -m "Restarting CI without cache: /nocache". This should cause a CI run without a cache.

  • Tom Aarsen

@tomaarsen
Copy link
Member Author

@tomaarsen tomaarsen commented Dec 10, 2021

I believe this works sufficiently. I'm going to try it out on #2908.

@tomaarsen tomaarsen merged commit c075dab into nltk:develop Dec 10, 2021
16 checks passed
@tomaarsen tomaarsen deleted the feature/no-cache-CI branch Dec 10, 2021
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.

None yet

1 participant