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)を10個のファイルに適用-7 #7316

Merged
merged 3 commits into from
Feb 21, 2024

Conversation

masyuko0222
Copy link
Contributor

Issue

概要

現在Style/HashSyntaxが適用されていないファイルがいくつかあります。
そのうち下記10個のファイルを適用しました。

    - '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'

変更確認方法

  1. feature/apply-rubocop-rule-7をローカルに取り込む
  2. rubocopを実行し、警告が出ないことを確認する

Screenshot

画面確認はございません。

@masyuko0222 masyuko0222 marked this pull request as ready for review February 6, 2024 00:30
@masyuko0222
Copy link
Contributor Author

@kurumadaisuke
お疲れ様です。
ご都合よろしければ、こちらのPRレビューしていただきたいです🙇‍♂️

@kurumadaisuke
Copy link
Contributor

@masyuko0222
かしこまりました!!
こちらレビューさせていただきます🙏
少々お時間いただくと思いますがよろしくお願いします🙏

@masyuko0222 masyuko0222 changed the title Style/HashSyntaxを10個のファイルに適用 Style/HashSyntaxを10個のファイルに適用-7 Feb 6, 2024
@masyuko0222 masyuko0222 self-assigned this Feb 6, 2024
@masyuko0222 masyuko0222 changed the title Style/HashSyntaxを10個のファイルに適用-7 Rubocopのルール(Style/HashSyntax)を10個のファイルに適用-7 Feb 6, 2024
Copy link
Contributor

@kurumadaisuke kurumadaisuke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@masyuko0222
お待たせしました🙏
問題なくrubocop通ったことを確認しましたのでApprove致します!!
お疲れ様でした🙏
スクリーンショット 2024-02-08 20 48 22

@masyuko0222
Copy link
Contributor Author

@kurumadaisuke
ありがとうございます!

@komagata
Approveいただいたので、レビューよろしくお願いいたします!

@masyuko0222 masyuko0222 requested a review from komagata February 8, 2024 12:44
@komagata
Copy link
Member

@masyuko0222 コンフリクトの解消をお願いします〜

@masyuko0222 masyuko0222 force-pushed the feature/apply-rubocop-rule-7 branch from 8f575c7 to 674a91f Compare February 13, 2024 11:25
@masyuko0222
Copy link
Contributor Author

@komagata
コンフリクトを直したのでご確認お願いいたします。

@komagata
Copy link
Member

komagata commented Feb 15, 2024

@masyuko0222 別のconflictが発生したようなので修正お願いします〜

@masyuko0222 masyuko0222 force-pushed the feature/apply-rubocop-rule-7 branch from 674a91f to 71d36a8 Compare February 15, 2024 10:47
@masyuko0222
Copy link
Contributor Author

@komagata
コンフリクトの解消をいたしました、ご確認お願いします。
それと、にしたつさんにメンション送ってしまっていますー👀

@komagata
Copy link
Member

@masyuko0222 テストが通ってからのレビュー依頼をお願いします〜

@masyuko0222 masyuko0222 force-pushed the feature/apply-rubocop-rule-7 branch from 71d36a8 to dc9ba1f Compare February 16, 2024 10:57
@masyuko0222
Copy link
Contributor Author

@komagata
度々失礼しましたー。レビューのほどよろしくお願いいたします。

@masyuko0222 masyuko0222 requested review from komagata and removed request for komagata February 18, 2024 01:51
@masyuko0222 masyuko0222 force-pushed the feature/apply-rubocop-rule-7 branch from 4c417a0 to a8f83e5 Compare February 19, 2024 11:07
@@ -11,15 +11,3 @@
# Configuration parameters: EnforcedStyle, EnforcedShorthandSyntax, UseHashRocketsWithSymbolValues, PreferHashRocketsForNonAlnumEndingSymbols.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

残りがコメントだけになってるのでファイル自体の削除をお願いします〜

@masyuko0222
Copy link
Contributor Author

masyuko0222 commented Feb 20, 2024

@komagata
何度もすみません。
改めてよろしくお願いしますー

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit d0089b4 into main Feb 21, 2024
2 checks passed
@komagata komagata deleted the feature/apply-rubocop-rule-7 branch February 21, 2024 12:57
@github-actions github-actions bot mentioned this pull request Feb 21, 2024
19 tasks
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.

3 participants