Skip to content
This repository has been archived by the owner on Jan 19, 2023. It is now read-only.

Re-add npm cache #2645

Closed
GuessWhoSamFoo opened this issue Jul 13, 2021 · 0 comments · Fixed by #2743
Closed

Re-add npm cache #2645

GuessWhoSamFoo opened this issue Jul 13, 2021 · 0 comments · Fixed by #2743
Labels
ci/cd enhancement New feature or request
Projects
Backlog

Comments

@GuessWhoSamFoo
Copy link
Contributor

Describe the problem/challenge you have
The npm cache was removed in #2065 to debug npm related build errors. This has since been resolved.

Describe the solution you'd like
We could revert the old config. The setup-node action now supports caching but only in the root directory.
xref actions/setup-node#275
@GuessWhoSamFoo GuessWhoSamFoo added enhancement New feature or request ci/cd labels Jul 13, 2021
@wwitzel3 wwitzel3 added this to Incoming in Backlog via automation Jul 15, 2021
@wwitzel3 wwitzel3 moved this from Incoming to Ecosystem in Backlog Jul 15, 2021
@GuessWhoSamFoo GuessWhoSamFoo mentioned this issue Aug 9, 2021
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
ci/cd enhancement New feature or request
Projects
Backlog
Ecosystem
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add npm cache GuessWhoSamFoo/octant
1 participant
@GuessWhoSamFoo