rejuvenate / lovelace-horizon-card

Sun Card successor: Visualize the position of the Sun over the horizon.

MIT License

474 stars 35 forks source link

Use current Node and Yarn version in PR workflow #153

Closed scinos closed 2 weeks ago

scinos commented 2 weeks ago

Pull Request Template for Home Assistant / Lovelace Card Repository

Overview

Fixes the CI errors introduced in #152

Type of Change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[ ] Documentation update
[ ] Refactoring
[ ] Other (please describe):

Additional Details

The fix consist in a few changes:

Update actions/setup-node to latest version.
Use native support for caching in actions/setup-node so we don't to deal with the cache ourselves.
Be sure it uses Node version from .nvmrc.
Enable corepack, so we pick yarn version from package.json.

Tested locally with act, seems to work fine. Will keep an eye on GH Actions results once this get approved.

scinos commented 2 weeks ago

Closing this so I can generate it from a branch, instead of using my fork's main.