Skip to content

ci: fix node modules cache doesn't work#16881

Merged
pissang merged 1 commit into
masterfrom
ci-cache
Apr 14, 2022
Merged

ci: fix node modules cache doesn't work#16881
pissang merged 1 commit into
masterfrom
ci-cache

Conversation

@plainheart
Copy link
Copy Markdown
Member

@plainheart plainheart commented Apr 13, 2022

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

Fix node modules cache doesn't work for missing the step id and the wrong cache target directory.

See https://github.com/apache/echarts/actions/runs/2162264848

@echarts-bot
Copy link
Copy Markdown

echarts-bot Bot commented Apr 13, 2022

Thanks for your contribution!
The community will review it ASAP. In the meanwhile, please checkout the coding standard and Wiki about How to make a pull request.

The pull request is marked to be PR: author is committer because you are a committer of this project.

@pissang pissang merged commit b2168d5 into master Apr 14, 2022
@echarts-bot
Copy link
Copy Markdown

echarts-bot Bot commented Apr 14, 2022

Congratulations! Your PR has been merged. Thanks for your contribution! 👍

@pissang pissang deleted the ci-cache branch April 14, 2022 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants