-
Notifications
You must be signed in to change notification settings - Fork 0
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
コーディング規約についてのページを追加 #1449
The head ref may contain hidden characters: "feature/\u30B3\u30FC\u30C7\u30A3\u30F3\u30B0\u898F\u7D04\u306B\u3064\u3044\u3066\u306E\u30DA\u30FC\u30B8\u3092\u8FFD\u52A0"
コーディング規約についてのページを追加 #1449
Conversation
|
||
- Vue.js アプリケーション | ||
- TypeScript | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Java側と比べると、リンクだけが記載されている状況なので、一言だけでも説明や特徴が書いてあるとバランスが良いかと思います
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TakashiOhkubo
ご指摘いただきありがとうございます。
Java アプリケーションとVue.jsアプリケーションで、
記載の粒度が揃うように段組みを変更のうえ追記いたしましたので、
再度ご確認お願いいたします。
Maris側のドキュメントへの反映は別途実施いたします。
この Pull request で実施したこと
Marisのコーディング規約の階層と内容と揃うように、コーディング規約のページを追加しました。
該当ページの.NETアプリケーションに関する記述を、Javaアプリケーションに関する記述へと置き換えました。
この Pull request では実施していないこと
フロントエンド側については手を入れておりません。
TypeScriptの規約のリンク先となっているAirbnb JavaScript Style Guide()にについて、
正確にはAirbnb JavaScript Style GuideをTypeScript用にカスタマイズしたライブラリをVue用にカスタマイズしたライブラリ
を使用している状態ですが、
この点についてはややこしくなることに加えて ES Lint 9 へのバージョンアップで大きく変わる可能性が高いため、修正しておりません。
package.json上で依存関係が見える、
Vue用にカスタマイズしたライブラリ
をリンク先にしておくほうが適切かもしれないとは考えており、
この点ご意見いただけると助かります。
Issues や Discussions 、関連する Web サイトなどへのリンク
Javaに関して記載すべき内容のコメント