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

ci(workflow): add cache to workflows using actions/setup-node #407

Closed
wants to merge 1 commit into from

Conversation

oscard0m
Copy link
Member

Description

Add cache to workflows using actions/setup-node

Context

setup-node GitHub Action just released a new option to add cache to steps using it.

You can find the details here: https://github.blog/changelog/2021-07-02-github-actions-setup-node-now-supports-dependency-caching/


🤖 This PR has been generated automatically by this octoherd script, feel free to run it in your GitHub user/org repositories! 💪🏾

id: openapi_types
- run: npm run update-endpoints
if: github.event_name == 'repository_dispatch' && github.event.action == 'octokit/types.ts release'
if: github.event_name == 'repository_dispatch' && github.event.action ==
'octokit/types.ts release'
Copy link
Contributor

Choose a reason for hiding this comment

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

@oscard0m can you revert the style changes? Odd that this happened, don't you use prettier with default settings for formatting?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep, I will revert them


I think I'm running prettier with no options so the default ones are applied but I will check what's happening.

Copy link
Member Author

Choose a reason for hiding this comment

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

Found the issue, latest changes were not released in my octoherd script. I just run it in my fork and it worked: https://github.com/oscard0m/plugin-enterprise-server.js/pull/2

Copy link
Member Author

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants