Skip to content

Commit

Permalink
[docs] Clear .next cache
Browse files Browse the repository at this point in the history
  • Loading branch information
@oliviertassinari
oliviertassinari committed Dec 17, 2020
1 parent 2a473d1 commit f96b45f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion docs/package.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
"build": "cross-env NODE_ENV=production next build --profile",
"build:clean": "rimraf .next && yarn build",
"build-sw": "node ./scripts/buildServiceWorker.js",
"dev": "rimraf ./node_modules/.cache/babel-loader && next dev",
"dev": "rimraf ./node_modules/.cache/babel-loader && rimraf ./.next/cache/ && next dev",

This comment has been minimized.

Sign in to view

Copy link
@eps1lon

eps1lon Dec 18, 2020

Member

This should be a separate command. You're slowing down dev considerably for for the very rare bug.

This comment has been minimized.

Sign in to view

Copy link
@oliviertassinari

oliviertassinari Dec 18, 2020 •
edited
Loading

Author Member

OK, reverting then, we can ignore. I didn't realize it would slow down dev significantly.

This comment has been minimized.

Sign in to view

Copy link
@eps1lon

eps1lon Dec 18, 2020

Member

I didn't even realize we cleared the babel cache. That's really odd. If there's a bug we should report it to next instead. If clearing cache is required for dev to work then they should do it as part of next dev where they can be more clever about what to invalidate.

This comment has been minimized.

Sign in to view

Copy link
@oliviertassinari

oliviertassinari Dec 18, 2020 •
edited
Loading

Author Member

Clearing the babel cache is required for kentcdodds/babel-plugin-preval#19 (comment)

This comment has been minimized.

Sign in to view

Copy link
@eps1lon

eps1lon Dec 18, 2020

Member

Gotcha, thanks. I have it on my TODO list after we were no longer able to upgrade. Might be a good idea after all to revisit it if we can solve two problems at once.

This comment has been minimized.

Sign in to view

Copy link
@oliviertassinari

oliviertassinari Dec 18, 2020

Author Member

Haha, I have personally removed it from my TODO list, but I can get the idea that being on an older version can create an increasing amount of debt, if for some reason, in 1 year it blocks us.
"deploy": "git push material-ui-docs next:next",
"export": "rimraf docs/export && next export --threads=3 -o export && yarn build-sw",
"icons": "rimraf public/static/icons/* && node ./scripts/buildIcons.js",
Expand Down

0 comments on commit f96b45f

Please sign in to comment.