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

🗑️ Log を削除 #112

Merged
merged 1 commit into from
Dec 4, 2023
Merged

🗑️ Log を削除 #112

merged 1 commit into from
Dec 4, 2023

Conversation

tatsutakein
Copy link
Member

@tatsutakein tatsutakein commented Dec 4, 2023

Issue

  • close #ISSUE_NUMBER 🦕

概要

Log を削除します。

レビュー観点

特になし

レビューレベル

  • Lv0: まったく見ないで Approve する
  • Lv1: ぱっとみて違和感がないかチェックして Approve する
  • Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
  • Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

  • すぐに見てもらいたい ( hotfix など ) 🚀
  • 今日中に見てもらいたい 🚗
  • 今日〜明日中で見てもらいたい 🚶
  • 数日以内で見てもらいたい 🐢

参考リンク

スクリーンショット

Before After

Summary by CodeRabbit

  • リファクタ
    • ログ機能を削除し、コードの整理を行いました。これにより、アプリのパフォーマンスが向上しました。

@tatsutakein tatsutakein requested a review from a team as a code owner December 4, 2023 11:27
Copy link

coderabbitai bot commented Dec 4, 2023

Walkthrough

ログ機能の削除が行われた。具体的には、Loggerのインポート文とlogプロパティの初期化、そしてログ出力のコードが複数のKotlinファイルから取り除かれた。これにより、コードはよりシンプルになり、ログに関連するオーバーヘッドが削減された。

Changes

ファイルパス 変更内容
.../core/data/.../OfflineFirstScheduleRepository.kt Loggerの初期化とログ出力コードを削除
.../core/domain/.../GetRecentScheduleUseCase.kt Loggerのインポートとlogプロパティ、ログ出力コードを削除
.../core/network/.../SupabaseScheduleRemoteDataSource.kt Loggerのインポートとlogプロパティを削除

🐰✨
コードからログが消えた日
シンプルさが戻り、静けさが訪れた
デバッグの兎、静かに跳ねる
新たな始まりを告げる朝日に向かって 🌅🚀


Tips

Chat with CodeRabbit Bot (@coderabbitai)

  • If you reply to a review comment from CodeRabbit, the bot will automatically respond.
  • To engage with CodeRabbit bot directly around the specific lines of code in the PR, mention @coderabbitai in your review comment
  • Note: Review comments are made on code diffs or files, not on the PR overview.
  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Commands (invoked as PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai help to get help.

Note: For conversation with the bot, please use the review comments on code diffs or files.

CodeRabbit Configration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • The JSON schema for the configuration file is available here.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/coderabbit-overrides.v2.json

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Review Status

Actionable comments generated: 0

Configuration used: CodeRabbit UI

Commits Files that changed from the base of the PR and between 3f2ea9b and c864c10.
Files selected for processing (3)
  • core/data/src/commonMain/kotlin/club/nito/core/data/OfflineFirstScheduleRepository.kt (2 hunks)
  • core/domain/src/commonMain/kotlin/club/nito/core/domain/GetRecentScheduleUseCase.kt (3 hunks)
  • core/network/src/commonMain/kotlin/club/nito/core/network/schedule/SupabaseScheduleRemoteDataSource.kt (3 hunks)
Additional comments: 7
core/data/src/commonMain/kotlin/club/nito/core/data/OfflineFirstScheduleRepository.kt (1)
  • 3-15: > Note: This review was outside the patches, so it was mapped to the patch with the greatest overlap. Original lines [3-29]

コードレビューを行いましたが、変更された部分に問題は見つかりませんでした。ログ機能の削除が目的とされており、その目的に沿った変更が行われていることを確認しました。

core/domain/src/commonMain/kotlin/club/nito/core/domain/GetRecentScheduleUseCase.kt (3)
  • 9-14: 変更されたインポートステートメントは、ログ機能の削除に関連しているわけではないため、この部分はPRの目的と一致しています。

  • 25-30: GetRecentScheduleExecutorクラスのinvokeメソッドからログ記述が削除されており、PRの目的に沿っています。

  • 51-56: invokeメソッドの終わりとtransformToParticipantScheduleメソッドにログ記述がないことを確認しました。これもPRの目的に一致しています。

core/network/src/commonMain/kotlin/club/nito/core/network/schedule/SupabaseScheduleRemoteDataSource.kt (3)
  • 5-10: ログ機能に関連するインポート文が削除されています。これはPRの目的に沿っています。

  • 18-22: SupabaseScheduleRemoteDataSource クラスからログプロパティが削除され、ログ機能がなくなりました。これもPRの目的に沿っています。

  • 36-39: ログ機能の削除による変更はここには表示されていませんが、これはPRの目的に沿っていると考えられます。

@tatsutakein tatsutakein merged commit 6114350 into main Dec 4, 2023
4 checks passed
@tatsutakein tatsutakein deleted the rt/remove-log branch December 4, 2023 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant