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

PropsでcamelCaseを使い,省略記法を強制するように変更 #2070

Merged
merged 8 commits into from
Jun 3, 2024

Conversation

cm-ayf
Copy link
Contributor

@cm-ayf cm-ayf commented May 8, 2024

内容

propsにcamelCaseを使うようにします.そのために,今までpropsをkebab-caseにしていたルールvue/attribute-hyphenationの設定を変更し,適用しました.

また,同じIssueで言及されているように,これによってv-bind:の省略記法が利用できるようになります.省略記法の利用を強制するルールを作成し,適用しました.

関連 Issue

closes: #1945

スクリーンショット・動画など

スクリーンショット 2024-05-08 17 09 40 スクリーンショット 2024-05-08 17 10 01

@cm-ayf cm-ayf requested a review from a team as a code owner May 8, 2024 08:10
@cm-ayf cm-ayf requested review from Hiroshiba and removed request for a team May 8, 2024 08:10
@Hiroshiba
Copy link
Member

Hiroshiba commented May 13, 2024

@cm-ayf 変更ありがとうございます!!

もしやと思って調べてみたのですが、もしかしたらv-bindのshorthandのESLintも既存かも・・・?
https://eslint.vuejs.org/rules/v-bind-style

というかこれrecommendedに追加されてるはずですね。。
と思ってeslint-vue-pluginのバージョン調べてみたら2年前のが今入ってますね。。。。。

・・・eslint-vue-pluginを全部最新のにしちゃうとかどうでしょう!!!

(とりあえず一旦頂いたフルリクエストをマージさせていただいてから変更、という形でももちろん大丈夫です!!)

@cm-ayf
Copy link
Contributor Author

cm-ayf commented May 13, 2024

2年前ともなるとeslint自体などと互換性があるかちょっと怪しく、かなり面倒そうですので、別PR案を推します。

@Hiroshiba
Copy link
Member

@cm-ayf たしかにです・・・!
先に本PRをマージさせていただいてから追撃する形もありなのかなと感じました!

もう1点すみません、v-on系のpropsが変更されていないことに気づきました!
https://github.com/VOICEVOX/voicevox/pull/2070/files#diff-0cfb845b6a63c022cc5a25632bda227491166d6a8e76113a494783996a7b0f34L123-L128

こちらも一緒にやってしまうか、PRのタイトルを変更して辻褄を合わせるか、どちらかお願いできれば・・・!

ちょっと調べてみた感じv-on系のhyphenationを制御するESLint ruleももしかしたらあるかもでした。世の中進んでますね。。。
https://eslint.vuejs.org/rules/v-on-event-hyphenation

@Hiroshiba
Copy link
Member

ちょっと手元で最新版のeslintとかを試してみたんですが、一切変更が入リませんでした!

公式のeslint ruleを使って独自プラグインの数を減らすとメンテナンス性が上がるのかなと思います。
ということで公式の使う形でお願いできると・・・!! @cm-ayf

@Hiroshiba
Copy link
Member

@cm-ayf 👀

@cm-ayf
Copy link
Contributor Author

cm-ayf commented Jun 1, 2024

多忙により、PRを断念し、あるいは他の方へ引き継ぎたいです……

@Hiroshiba
Copy link
Member

@cm-ayf 承知しました! こちらで巻き取るので後ほどレビューお願いします!!

最新版の ESLint に上げて、公式にある機能は公式の機能を利用したいと思います!

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

@cm-ayf 実装してみました!!

まあまあいい感じかなぁ。しばらくちょっとこれでやってみてどうしてもダメそうだったら戻せばよさそう!

@cm-ayf
Copy link
Contributor Author

cm-ayf commented Jun 3, 2024

問題なさそうに見えます(PRのauthorはレビュー付けられないんですよね)

@Hiroshiba
Copy link
Member

確認ありがとうございます、マージします!

@Hiroshiba Hiroshiba merged commit b0c800e into VOICEVOX:main Jun 3, 2024
9 checks passed
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.

<template>内に書くprop名をkebab-caseからcamelCaseに変えたい
2 participants