fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
281 stars 71 forks source link

rubocopのルール(Style/HashSyntax)を適用する1 #7280

Closed komagata closed 7 months ago

komagata commented 8 months ago

現状は下記でrubocopのStyle/HashSyntaxのルールを特定のファイルに対しては除外している。 https://github.com/fjordllc/bootcamp/blob/3074feb17edfa0b5f1813a091712db2705d1aeb3/.rubocop_todo.yml#L15

これをやめたい。

全部のファイルを一気に除外をなくすと大変なので10ファイルずつお願いします。(最後の一人は10ファイルにみたいないことになるかもしれないですが、それは構いません)

kyokucho1989 commented 8 months ago

@nishitatsu-dev

お疲れ様です。同じ種類のissueを担当しています。

有効にするファイルが重複してしまうとよくないので、予めここに書き込みます。 私は上から11番目から10ファイル分有効にしています。

以下の通りです。


    - 'app/controllers/notifications/read_by_category_controller.rb'
     - 'app/controllers/notifications_controller.rb'
     - 'app/controllers/products_controller.rb'
     - 'app/controllers/user_sessions_controller.rb'
     - 'app/helpers/meta_tags_helper.rb'
     - 'app/helpers/page_tab_helper.rb'
     - 'app/helpers/reactions_helper.rb'
     - 'app/jobs/ai_answer_create_job.rb'
     - 'app/mailers/activity_mailer.rb'
     - 'app/mailers/notification_mailer.rb'
reckyy commented 8 months ago

@nishitatsu-dev お疲れ様です。私も同種Issueを担当することになりました。 上から21番目以降10行を有効にします。 .rubocop_todo.ymlだと、36〜45行です。

    - 'app/models/announcement_notifier.rb'
    - 'app/models/answer.rb'
    - 'app/models/answer_notifier.rb'
    - 'app/models/card.rb'
    - 'app/models/chat_notifier.rb'
    - 'app/models/check_callbacks.rb'
    - 'app/models/comment.rb'
    - 'app/models/comment/after_create_callback.rb'
    - 'app/models/concerns/mentioner.rb'
    - 'app/models/concerns/reactionable.rb'
nishitatsu-dev commented 8 months ago

@kyokucho1989, @reckyy ご連絡ありがとうございます🙏 では、最初の10ファイルを担当します〜

    - 'app/controllers/admin/campaigns_controller.rb'
    - 'app/controllers/announcements_controller.rb'
    - 'app/controllers/api/admin/count_controller.rb'
    - 'app/controllers/api/bookmarks_controller.rb'
    - 'app/controllers/api/checks_controller.rb'
    - 'app/controllers/api/followings_controller.rb'
    - 'app/controllers/api/products/unchecked_controller.rb'
    - 'app/controllers/api/session_controller.rb'
    - 'app/controllers/api/watches/toggle_controller.rb'
    - 'app/controllers/application_controller.rb'
reckyy commented 8 months ago

41〜50つ目のファイルを担当します。 .rubocop_todo.ymlだと、56~65行目です。

- 'app/models/page_notifier.rb'
- 'app/models/practice.rb'
- 'app/models/product.rb'
- 'app/models/product_callbacks.rb'
- 'app/models/product_notifier_for_colleague.rb'
- 'app/models/product_notifier_for_practice_watcher.rb'
- 'app/models/question_callbacks.rb'
- 'app/models/reaction.rb'
- 'app/models/regular_event/participants_creator.rb'
- 'app/models/regular_event/participants_watcher.rb'
kurumadaisuke commented 8 months ago

私も同じissueを担当したので .rubocop_todo.ymlだと、66~75行目担当します。

    - 'app/models/regular_event_update_notifier.rb'
    - 'app/models/report.rb'
    - 'app/models/report_notifier.rb'
    - 'app/models/searcher.rb'
    - 'app/models/subscription.rb'
    - 'app/models/unfinished_data_destroyer.rb'
    - 'app/models/user.rb'
    - 'app/notifiers/activity_notifier.rb'
    - 'app/notifiers/discord_notifier.rb'
    - 'lib/bootcamp/setup.rb'
masyuko0222 commented 8 months ago

.rubocop_todo.ymlの、76~85行目を担当します。

    - 'test/deliveries/activity_delivery_test.rb'
    - 'test/mailers/activity_mailer_test.rb'
    - 'test/mailers/notification_mailer_test.rb'
    - 'test/mailers/previews/activity_mailer_preview.rb'
    - 'test/mailers/previews/notification_mailer_preview.rb'
    - 'test/models/answer_test.rb'
    - 'test/models/bookmark_test.rb'
    - 'test/models/category_test.rb'
    - 'test/models/discord/times_channel_test.rb'
    - 'test/models/event_test.rb'
SuzukaHori commented 8 months ago

.rubocop_todo.ymlの以下のファイルを担当します。

    - 'test/models/product_test.rb'
    - 'test/models/question_test.rb'
    - 'test/models/regular_event/participants_creator.rb'
    - 'test/models/regular_event/participants_watcher_test.rb'
    - 'test/models/regular_event_test.rb'
    - 'test/models/user_test.rb'
    - 'test/notifiers/discord_notifier_test.rb'
    - 'test/supports/api_helper.rb'
    - 'test/supports/tag_helper.rb'
    - 'test/system/announcements_test.rb'

最初の方の変更がマージされているため、ローカルでは行数が異なりますが、元のファイルだと86~95行目に当たると思います。

masyuko0222 commented 8 months ago

.rubocop_todo.ymlの、96~105行目を担当します。

    - 'test/system/company/products_test.rb'
    - 'test/system/current_user/products_test.rb'
    - 'test/system/notification/products_test.rb'
    - 'test/system/notification/questions_test.rb'
    - 'test/system/notification/signed_up_test.rb'
    - 'test/system/product/self_assigned_test.rb'
    - 'test/system/product/unchecked_test.rb'
    - 'test/system/sign_up_test.rb'
    - 'test/system/user/products_test.rb'
    - 'test/test_helper.rb'
SuzukaHori commented 8 months ago

以下のファイルを担当します。 元の.rubocop_todo.ymlだと、46~55行目に当たります。

    - 'app/models/concerns/searchable.rb'
    - 'app/models/correct_answer_notifier.rb'
    - 'app/models/discord/server.rb'
    - 'app/models/event.rb'
    - 'app/models/first_report_notifier.rb'
    - 'app/models/grass.rb'
    - 'app/models/learning.rb'
    - 'app/models/notification.rb'
    - 'app/models/notification_facade.rb'
    - 'app/models/notifier_to_watching_user.rb'
nishitatsu-dev commented 7 months ago

リリース後のmainブランチで確認取れましたので、closeします