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

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

Closed
komagata opened this issue Jan 31, 2024 · 10 comments
Closed

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

komagata opened this issue Jan 31, 2024 · 10 comments
Assignees

Comments

@komagata
Copy link
Member

komagata commented Jan 31, 2024

現状は下記でrubocopのStyle/HashSyntaxのルールを特定のファイルに対しては除外している。

Exclude:

これをやめたい。

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

@kyokucho1989
Copy link
Contributor

@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
Copy link
Contributor

reckyy commented Feb 1, 2024

@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
Copy link
Contributor

@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
Copy link
Contributor

reckyy commented Feb 2, 2024

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
Copy link
Contributor

私も同じ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
Copy link
Contributor

.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
Copy link
Contributor

.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
Copy link
Contributor

masyuko0222 commented Feb 6, 2024

.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
Copy link
Contributor

以下のファイルを担当します。
元の.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
Copy link
Contributor

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

@komagata komagata moved this to 完成 in bootcamp Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

7 participants