fix: allow npm install without changing yarn-first workflow#1229
fix: allow npm install without changing yarn-first workflow#1229JackWilb wants to merge 10 commits into
Conversation
|
Starting deployment of preview ⏳... |
|
@copilot remove p_DO_NOT_USE_NPM_USE_YARN and fix Run yarn lint /home/runner/work/study/study/src/analysis/individualStudy/summary/utils.tsx /home/runner/work/study/study/src/components/audioAnalysis/AudioProvenanceVis.tsx /home/runner/work/study/study/src/components/audioAnalysis/useUpdateProvenance.tsx /home/runner/work/study/study/src/controllers/VegaController.tsx /home/runner/work/study/study/src/parser/types.ts /home/runner/work/study/study/src/store/store.tsx ✖ 7 problems (7 errors, 0 warnings) error Command failed with exit code 1. |
Implemented and pushed the requested lint fixes and cleanup in commit |
…or circular reference
Closes #1218
Summary
Allow habitual
npm installusers to install dependencies without changing the project's Yarn-first workflow.Changes
package.json: remove the Yarn-onlypreinstallguard that hard-failed npm installs.package.json: align lint dependencies for npm peer resolution by downgradingeslintto^8.57.1andeslint-plugin-react-hooksto^4.6.2while keeping the Airbnb-based lint setup intact.package.json: update@visdesignlab/upset2-reactto^1.2.4,react-vegato^8.0.0, andvega-liteto^6.0.1so strict npm peer resolution succeeds.yarn.lock: refresh the Yarn lockfile to reflect the dependency updates.package-lock.json: add an npm lockfile generated from the updated dependency graph.Validation
yarn install --mode=skip-buildnpm install --package-lock-only --ignore-scriptsyarn typecheck(currently fails on pre-existing errors insrc/store/hooks/tests/usePreviousStep.spec.ts)