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

chore(gatsby): move cache folders to .cache #25192

Merged
merged 1 commit into from
Jun 22, 2020

Conversation

wardpeet
Copy link
Contributor

@wardpeet wardpeet commented Jun 22, 2020

Description

Mover terser & babel cache into .cache folder instead of default node_modules location. This helps to make sure changes like #15601 actually get applied.

https://twitter.com/wardpeet/status/1266504690704973824

Related Issues

Related to #5880 (I'll comment and close that one)

@wardpeet wardpeet requested a review from a team as a code owner June 22, 2020 11:21
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Jun 22, 2020
Copy link
Contributor

@sidharthachatterjee sidharthachatterjee left a comment

Choose a reason for hiding this comment

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

Ah, this is great cause gatsby clean would handle these cases. I've hit them a few times recently while working on #25120

@sidharthachatterjee sidharthachatterjee added bot: merge on green Gatsbot will merge these PRs automatically when all tests passes and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Jun 22, 2020
@gatsbybot gatsbybot merged commit 94d1c49 into gatsbyjs:master Jun 22, 2020
@wardpeet wardpeet deleted the fix/cache-dirs-babel-terser branch June 22, 2020 13:31
pragmaticpat pushed a commit to pragmaticpat/gatsby that referenced this pull request Apr 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: merge on green Gatsbot will merge these PRs automatically when all tests passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants