Skip to content

eslint-plugin-storybookがインストールされているのに効いてなかったので有効化#299

Open
Masa-Shin wants to merge 5 commits intomasterfrom
0/feature/enable-eslint-plugin-storybook
Open

eslint-plugin-storybookがインストールされているのに効いてなかったので有効化#299
Masa-Shin wants to merge 5 commits intomasterfrom
0/feature/enable-eslint-plugin-storybook

Conversation

@Masa-Shin
Copy link
Copy Markdown
Contributor

  • eslint-plugin-storybookを有効化
  • commitlintの100文字制限は意味がわからないので無効化
  • Storybookをマイナーアップデート

@Masa-Shin Masa-Shin self-assigned this Sep 4, 2025
@Masa-Shin Masa-Shin force-pushed the 0/feature/enable-eslint-plugin-storybook branch 2 times, most recently from 4a3e056 to dc4e3a5 Compare September 4, 2025 10:33
Comment thread .eslintignore
@@ -0,0 +1 @@
!.storybook
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Comment thread package.json
"@storybook/core": "^8.3.5",
"@storybook/vue3-vite": "^8.3.5",
"@types/body-scroll-lock": "^3.1.2",
"@types/eslint__js": "^8.42.3",
Copy link
Copy Markdown
Contributor Author

@Masa-Shin Masa-Shin Sep 5, 2025

Choose a reason for hiding this comment

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

型は@eslint/jsに入ってるので不要
https://www.npmjs.com/package/@types/eslint__js

Comment thread package.json
"extends": [
"plugin:storybook/recommended"
]
},
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ここだと効果がない模様

Comment thread commitlint.config.js
rules: {
'body-max-line-length': [0, 'always']
}
}
Copy link
Copy Markdown
Contributor Author

@Masa-Shin Masa-Shin Sep 5, 2025

Choose a reason for hiding this comment

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

commitの説明に参考URLを貼っただけで100文字は超過するのと、そもそもcommitに文字数制限は別にいらんやろと言うことで切りました

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

そうだそうだ!!

Comment thread .storybook/preview.ts
@@ -1,4 +1,4 @@
import type { Preview } from '@storybook/vue3'
import type { Preview } from '@storybook/vue3-vite'
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ほええ

Comment thread package.json Outdated
"eslint-config-prettier": "^9.1.0",
"eslint-plugin-prettier": "^5.2.1",
"eslint-plugin-storybook": "9.1.4",
"eslint-plugin-storybook": "^0.11.6",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

バージョン不整合で戻しました

Copy link
Copy Markdown
Member

@tktcorporation tktcorporation left a comment

Choose a reason for hiding this comment

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

特徳のしんさん

@tktcorporation tktcorporation force-pushed the 0/feature/enable-eslint-plugin-storybook branch from dc4e3a5 to 47da08c Compare September 5, 2025 07:28
@tktcorporation tktcorporation self-assigned this Mar 16, 2026
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