concord-consortium / codap

CODAP (Common Online Data Analysis Platform)
MIT License
94 stars 38 forks source link

chore: move lint cache files into `node_modules/.cache` folder #1402

Closed kswenson closed 4 weeks ago

kswenson commented 4 weeks ago

Recently, @nstclair-cc encountered an issue whereby lint:build failed with a module resolution error but all other commands (lint, build:webpack, test) succeeded. Seemed likely to be a caching issue but clean:cache didn't fix it. 🤔 Only later did I realize that the lint caches aren't cleared by clean:cache because they aren't stored in the node_modules/.cache folder with the other caches (e.g. webpack's). This PR fixes that so that the lint caches are in node_modules/.cache along with our other development-related caches, which means that the clean:cache command clears the lint caches as well.

Note that I've removed the old files from the .gitignore file, so that developers with existing repositories may encounter unversioned files named .eslintcache and .eslintcache.build in their workspace. These files can safely be deleted.

codecov[bot] commented 4 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.97%. Comparing base (564c4ec) to head (1662eef). Report is 3 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1402 +/- ## ======================================== Coverage 85.97% 85.97% ======================================== Files 533 533 Lines 26038 26038 Branches 7048 6656 -392 ======================================== Hits 22386 22386 - Misses 3374 3497 +123 + Partials 278 155 -123 ``` | [Flag](https://app.codecov.io/gh/concord-consortium/codap/pull/1402/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=concord-consortium) | Coverage Δ | | |---|---|---| | [cypress](https://app.codecov.io/gh/concord-consortium/codap/pull/1402/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=concord-consortium) | `72.59% <ø> (+<0.01%)` | :arrow_up: | | [jest](https://app.codecov.io/gh/concord-consortium/codap/pull/1402/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=concord-consortium) | `53.93% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=concord-consortium#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

cypress[bot] commented 4 weeks ago



Test summary

204 0 29 0Flakiness 1


Run details

Project codap-v3
Status Passed
Commit 78bafaa859
Started Aug 14, 2024 2:28 PM
Ended Aug 14, 2024 2:36 PM
Duration 08:27 💡
OS Linux Ubuntu -
Browser Chrome 127

View run in Cypress Cloud ➡️


Flakiness

cypress/e2e/toolbar.spec.ts Flakiness
1 codap toolbar > will open a slider

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Cloud