Skip to content

Comments

embedding cache steps by using @actions/cache#31

Merged
asottile merged 2 commits intopre-commit:masterfrom
trim21:master
Jun 13, 2020
Merged

embedding cache steps by using @actions/cache#31
asottile merged 2 commits intopre-commit:masterfrom
trim21:master

Conversation

@trim21
Copy link
Contributor

@trim21 trim21 commented Jun 8, 2020

a proposal, feel free to close it.

There is no need to add cache steps as @actions/cache is released and github actions can use it to cache files instead of adding a extra step.

Copy link
Member

@asottile asottile left a comment

Choose a reason for hiding this comment

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

seems good, this wasn't an option when it was originally written

please revert style changes and structure changes and run the pre-commit hooks (lol)

Copy link
Member

@asottile asottile left a comment

Choose a reason for hiding this comment

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

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.

2 participants