-
Notifications
You must be signed in to change notification settings - Fork 298
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
TypeScriptを更新 #959
TypeScriptを更新 #959
Conversation
"@vue/test-utils": "2.0.0-rc.6", | ||
"chai": "4.3.4", | ||
"chai-as-promised": "7.1.1", | ||
"electron": "19.0.6", | ||
"electron-devtools-installer": "3.2.0", | ||
"eslint": "6.8.0", | ||
"eslint": "8.24.0", | ||
"eslint-config-prettier": "8.5.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eslint-plugin-prettierのバージョンを更新したことで要求されるようになったので新しく追加しています。
prettier/eslint-plugin-prettier#253 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
これ、eslint-plugin-prettierを更新したわけではなく、@vue/eslint-config-prettierを更新した結果、依存しているeslint-plugin-prettierのバージョンが上がった、が正しかったです。
なんというか、すごくどっちでも良いですね・・・。 |
テスト通ってそうなのでDraft外します! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTMです!
マージのタイミングは @Hiroshiba さんにお任せします...!z
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM~
変更がコミットでわかりやすくてレビューしやすかったです、ありがとうございます!
またeslint周りの設定変更についてrefも助かりました〜
内容
TypeScriptのバージョンをv4.8.4に更新します。
それに伴い出た警告を解消するためにeslint系の依存関係も更新しています。
また、新たに出た型エラーやリンターエラーの修正も含まれます。
vue/multi-word-component-names だけまだ残っていますが、これに関しては @Hiroshiba さんの意見を伺いたいのであえて残しています。
個人的にはこのルールは設定で切ってしまってもいいかなと思っています。
関連 Issue