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

template内のコンポーネントの名称をPascalCaseに変更 #1874

Merged
merged 7 commits into from
Feb 26, 2024

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
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>内に書くコンポーネント名をkebab-caseからPascalCaseに変えたい
2 participants