Skip to content

Conversation

@cm-ayf
Copy link
Contributor

@cm-ayf cm-ayf commented Feb 25, 2024

内容

<template>内に書くコンポーネント名をkebab-caseからPascalCaseに変えました.
変更にあたっては,そのようなことを行ってくれるESLintルール https://eslint.vuejs.org/rules/component-name-in-template-casing を利用しました.

関連 Issue

closes: #1873

@cm-ayf cm-ayf force-pushed the component-name-in-template-casing branch from 79d775c to 3d76594 Compare February 25, 2024 14:45
@cm-ayf cm-ayf marked this pull request as ready for review February 25, 2024 14:46
@cm-ayf cm-ayf requested a review from a team as a code owner February 25, 2024 14:46
@cm-ayf cm-ayf requested review from Hiroshiba and removed request for a team February 25, 2024 14:46
@cm-ayf
Copy link
Contributor Author

cm-ayf commented Feb 25, 2024

テストが落ちてますが、コンポーネントがどこで定義されているか解決できてなさそうな感じです。vueとquasarに詳しい方から助言いただけると嬉しいです……

@Hiroshiba
Copy link
Member

@cm-ayf 設定変更したら動きました!(15分かかりました 😇 )

@Hiroshiba Hiroshiba changed the title Component name in template casing template内のコンポーネントの名称をPascalCaseに変更 Feb 25, 2024
@cm-ayf
Copy link
Contributor Author

cm-ayf commented Feb 25, 2024

ありがとうございます!

@cm-ayf
Copy link
Contributor Author

cm-ayf commented Feb 25, 2024

tests/unit/utils.ts:8に残ってますね

@cm-ayf
Copy link
Contributor Author

cm-ayf commented Feb 26, 2024

よさそうですね。
ところでこれをmergeするとconflictをばら撒くことになりますね……

@Hiroshiba
Copy link
Member

まあ差分も明確ですし、コンフリクトは別に良いかなと思いました!
特にDiscordで反対意見もなかったのでマージしたいと思います!
ありがとうございました!!またぜひ!!!

@Hiroshiba Hiroshiba merged commit d0fcc10 into VOICEVOX:main Feb 26, 2024
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>内に書くコンポーネント名をkebab-caseからPascalCaseに変えたい

2 participants