rejuvenate / lovelace-horizon-card

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

MIT License

476 stars 35 forks source link

Use corrent Node and Yarn version in PR workflow #154

Closed scinos closed 3 weeks ago

scinos commented 3 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.

ThomDietrich commented 3 weeks ago

Hey @scinos I went ahead and copied these changes to the releases workflow.