Skip to content

Conversation

@slaweet
Copy link
Contributor

@slaweet slaweet commented Jul 19, 2021

I lived for several months thinking that support for caching action on GHE is just a matter of time, because it's such an important thing to have. Only today, I discovered that originally it was not planned at all. And that people already created some workarounds. So I hope that linking the issue from the warning message will save other people from what happened to me :-)

@slaweet slaweet requested a review from a team as a code owner July 19, 2021 09:02
@brcrista
Copy link
Contributor

Thanks @slaweet -- I triggered CI and it looks like there is a linting issue. Not sure what it is, but could you give it a look?

slaweet added 2 commits July 19, 2021 19:58
I lived for several months thinking that support for caching action on GHE is just a matter of time, because it's such an important thing to have. Only today, I discovered that originally it was not planned at all. And that people already created some workarounds. So I hope that linking the issue from the warning message will save other people from what happened to me :-)
@slaweet
Copy link
Contributor Author

slaweet commented Jul 19, 2021

@brcrista thanks for the feedback. I fixed the lining error, amended the commit, and pushed the change. Please have a look again.

@brcrista
Copy link
Contributor

Fixed that error, something else failed. Run npm run build and commit the dist/index.js file to fix.

@slaweet
Copy link
Contributor Author

slaweet commented Jul 20, 2021

I run npm run build, which generated changes in dist/restore/index.js and dist/save/index.js. I committed and pushed those files.

I was so naive when I thought that if I'm just editing a string literal, I can do it in GitHub without checking out the repo 😄

Copy link
Contributor

@brcrista brcrista left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution! :shipit:

@brcrista brcrista merged commit 2ebdcff into actions:main Jul 20, 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.

2 participants