Replies: 1 comment 3 replies
-
|
ご連絡ありがとうございます。この指摘は、Wontfix を選択して無視しておいてくださいませ。 |
Beta Was this translation helpful? Give feedback.
3 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment

Uh oh!
There was an error while loading. Please reload this page.
-
@iamsin
Codeclimateより重複コードをリファクタリングするよう提案がありました。
https://codeclimate.com/github/sinProject-Inc/talk/pull/1446
以下の2つのファイルは処理内容がほとんど同じです。このクラスを作成された方は、役割が違うのでクラスを分けているのかと思いますのでそのままにしておいた方が良いと思います。2つのクラスを1つに統合すべきか、このCodeclimateのエラーは無視するべきかご意見いただけませんでしょうか。
src/lib/view/message.ts
src/lib/general/text_error.ts
Beta Was this translation helpful? Give feedback.
All reactions