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

npm update #372

Merged
merged 3 commits into from
Mar 3, 2021
Merged

npm update #372

merged 3 commits into from
Mar 3, 2021

Conversation

takano32
Copy link
Contributor

@takano32 takano32 commented Mar 3, 2021

#371 で大幅に package.json が書き換わったので npm update をして動作確認しました。

@takano32 takano32 requested a review from yuiseki March 3, 2021 05:42
@takano32
Copy link
Contributor Author

takano32 commented Mar 3, 2021

@takano32
Copy link
Contributor Author

takano32 commented Mar 3, 2021

babel-eslint 依存かもしれない。

see also. babel/babel-eslint#681

$ node node_modules/.bin/eslint --ext .js,.vue --ignore-path .gitignore .

TypeError: Cannot read property 'range' of null
Occurred while linting /Users/takano-mitsuhiro/GitHub/mapprint/components/PrintableMap.vue:113
    at SourceCode.getTokenBefore (/Users/takano-mitsuhiro/GitHub/mapprint/node_modules/eslint/lib/source-code/token-store/index.js:298:18)
    at checkSpacingBefore (/Users/takano-mitsuhiro/GitHub/mapprint/node_modules/eslint/lib/rules/template-curly-spacing.js:60:42)
    at TemplateElement (/Users/takano-mitsuhiro/GitHub/mapprint/node_modules/eslint/lib/rules/template-curly-spacing.js:119:17)
    at /Users/takano-mitsuhiro/GitHub/mapprint/node_modules/eslint/lib/linter/safe-emitter.js:45:58
    at Array.forEach (<anonymous>)
    at Object.emit (/Users/takano-mitsuhiro/GitHub/mapprint/node_modules/eslint/lib/linter/safe-emitter.js:45:38)
    at NodeEventGenerator.applySelector (/Users/takano-mitsuhiro/GitHub/mapprint/node_modules/eslint/lib/linter/node-event-generator.js:254:26)
    at NodeEventGenerator.applySelectors (/Users/takano-mitsuhiro/GitHub/mapprint/node_modules/eslint/lib/linter/node-event-generator.js:283:22)
    at NodeEventGenerator.enterNode (/Users/takano-mitsuhiro/GitHub/mapprint/node_modules/eslint/lib/linter/node-event-generator.js:297:14)
    at CodePathAnalyzer.enterNode (/Users/takano-mitsuhiro/GitHub/mapprint/node_modules/eslint/lib/linter/code-path-analysis/code-path-analyzer.js:634:23)

@yuiseki
Copy link
Collaborator

yuiseki commented Mar 3, 2021

ローカルではnpm update後もnpm run devやnpm run generateが動いてますか?eslintだけ落ちる状態?

@takano32
Copy link
Contributor Author

takano32 commented Mar 3, 2021

npm run dev は動作していましたが ESLint が落ちる状態でした。

package-lock.json をいちど消して npm install ののちに npm audit fix をしたところ、おおよそ正常なパッケージが入るようになったようです。

上記のコミットで npm run dev および npm run generate が動作すること、サイトが意図通りにレンダリングされていることの簡単な確認ができました。

Copy link
Collaborator

@yuiseki yuiseki left a comment

Choose a reason for hiding this comment

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

git checkout origin/npm-update
npm install
npm run generate
npm run dev

で動作確認できました。
LGTM!

@yuiseki yuiseki merged commit c697ccb into master Mar 3, 2021
@yuiseki yuiseki deleted the npm-update branch March 3, 2021 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants