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

ESLint関連パッケージを更新 #476

Merged
merged 19 commits into from
Jan 17, 2024

Conversation

munierujp
Copy link
Contributor

@munierujp munierujp commented Jan 9, 2024

概要 | About

ESLint関連パッケージを新しいバージョンに更新します。

  • @nuxtjs/eslint-config: 1.0.1 -> 8.0.0
    • 最新バージョンは12.0.0
  • @nuxtjs/eslint-module: 1.2.0 -> 4.1.0
    • 最新バージョンは4.1.0
  • eslint: 6.8.0 -> 7.32.0
    • 最新バージョンは8.56.0
  • eslint-plugin-html: 6.1.1 -> 7.1.0
    • 最新バージョンは7.1.0
  • eslint-plugin-nuxt: 2.0.0 -> 4.0.0
    • 最新バージョンは4.0.0
  • eslint-plugin-vue: 7.7.0 -> 9.19.2
    • 最新バージョンは9.19.2

なるべく新しいバージョンに更新していますが、更新により発生したエラーの解消が難しい場合は、それ以前のバージョンにとどめています。
既存コードでエラーが検知された場合、意図的なものなのかどうか判断つかないため、ひとまずコメントで無効化しています。

動作確認方法 | How to check

yarn lint

スクリーンショット | Screenshot

なし

@@ -9,6 +9,7 @@ try {
if (fs.existsSync("./nuxt-router-override.config.js")) {
router = require("./nuxt-router-override.config").default;
}
// eslint-disable-next-line no-empty
Copy link
Contributor Author

Choose a reason for hiding this comment

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

📝 単にfinallyブロックを削除してもよさそうですが、なんらかの意図があるかもしれないので無効化しています。

@@ -33,6 +33,7 @@ const _make_tilejson = (dir, base_name) => {

// load config
const list = require(`${CONFIG_DIR}list.json`)
// eslint-disable-next-line array-callback-return
Copy link
Contributor Author

Choose a reason for hiding this comment

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

📝 単にforEachに置き換えてもよさそうですが、なんらかの意図があるかもしれないので無効化しています。

@@ -248,10 +248,12 @@ export default {
self.mapConfig.layer_settings,
source.updated_search_key
);
// eslint-disable-next-line array-callback-return
Copy link
Contributor Author

Choose a reason for hiding this comment

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

📝 単にforEachに置き換えてもよさそうですが、なんらかの意図があるかもしれないので無効化しています。

markers.map((marker) => {
categories[marker.category] = true;
});
source.updated_at = updated_at;
// eslint-disable-next-line array-callback-return
Copy link
Contributor Author

Choose a reason for hiding this comment

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

📝 単にforEachに置き換えてもよさそうですが、なんらかの意図があるかもしれないので無効化しています。

Copy link
Member

@halsk halsk left a comment

Choose a reason for hiding this comment

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

LGTM

@halsk halsk merged commit 0af129b into codeforjapan:master Jan 17, 2024
4 checks passed
@munierujp munierujp deleted the update-eslint-packages branch January 17, 2024 04:09
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