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

fix(actions): attempt at fixing cache-dependency-path #412

Merged
merged 2 commits into from
Aug 15, 2023

Conversation

@chrisadubois chrisadubois requested a review from a team as a code owner August 14, 2023 20:54
@chrisadubois chrisadubois added the validated This pull request is validated for automation label Aug 14, 2023
Copy link
Contributor

@InteractiveTimmy InteractiveTimmy left a comment

Choose a reason for hiding this comment

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

the cache-dependency-path should be the path to the localized package-lock.json or yarn.lock file, the file at the value of this parameter is read by actions/setup-node@3, then the contents are processed similar to the native GitHub Actions hashFiles() call within actions/setup-node@3.

See actions/setup-node@3 documentation. Let me know if you disagree and believe these initial changes are correct.

See this related issue, it seems this was an API-breaking change they didn't account for.

Copy link
Contributor

@InteractiveTimmy InteractiveTimmy left a comment

Choose a reason for hiding this comment

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

LGTM per documentation of the consumed workflow.

@chrisadubois chrisadubois merged commit 7372b24 into momentum-design:main Aug 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
validated This pull request is validated for automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants