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

♻️ OptIn のアノテーション位置を変更 #104

Merged
merged 1 commit into from
Dec 2, 2023

Conversation

tatsutakein
Copy link
Member

@tatsutakein tatsutakein commented Dec 2, 2023

Issue

  • close #ISSUE_NUMBER 🦕

概要

OptIn のアノテーション位置を変更します。

レビュー観点

特になし

レビューレベル

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

レビュー優先度

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

参考リンク

スクリーンショット

Before After

Summary by CodeRabbit

  • リファクタ
    • Supabaseクライアントの生成方法を更新しました。
    • JSON設定に新しい命名戦略を追加しました。

@tatsutakein tatsutakein requested a review from a team as a code owner December 2, 2023 14:18
Copy link

coderabbitai bot commented Dec 2, 2023

Walkthrough

Kotlinファイルの変更点は、createNitoSupabaseClient関数から@OptIn(ExperimentalSerializationApi::class)アノテーションが削除され、createNitoKtorJsonSettings関数に新たにこのアノテーションが追加されたことです。さらに、createNitoKtorJsonSettings関数にはJsonNamingStrategy.SnakeCaseを値とする新しい設定namingStrategyが追加されました。

Changes

ファイル 変更前 変更後
.../network/SupabaseClient.kt createNitoSupabaseClient関数に@OptIn(ExperimentalSerializationApi::class)アノテーションがある。
createNitoKtorJsonSettings関数にnamingStrategy設定がない。
createNitoSupabaseClient関数から@OptIn(ExperimentalSerializationApi::class)アノテーションが削除された。
createNitoKtorJsonSettings関数に@OptIn(ExperimentalSerializationApi::class)アノテーションとnamingStrategy設定が追加された。

🐰💻

コードの森を駆け巡る

アノテーションは葉のように落ち

新たな設定が芽吹く春

スネークケースが道を照らす🌟


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 170eb8f and 5e1f2fb.
Files selected for processing (1)
  • core/network/src/commonMain/kotlin/club/nito/core/network/SupabaseClient.kt (2 hunks)
Additional comments: 2
core/network/src/commonMain/kotlin/club/nito/core/network/SupabaseClient.kt (2)
  • 25-26: コードの変更は問題なく、createNitoSupabaseClient関数から@OptIn(ExperimentalSerializationApi::class)アノテーションが削除されています。これにより、実験的なAPIの使用に関するコードの整理が行われていることが確認できます。

  • 39-40: createNitoKtorJsonSettings関数に@OptIn(ExperimentalSerializationApi::class)アノテーションが追加され、新しい設定パラメータnamingStrategyも導入されています。これにより、JSONの命名規則が変更され、シリアライゼーションの挙動に影響を与える可能性があります。依存するコードがこの変更に対応しているか確認する必要があります。

@tatsutakein tatsutakein merged commit 23f2264 into main Dec 2, 2023
4 checks passed
@tatsutakein tatsutakein deleted the rt/refactor-move-optin branch December 2, 2023 14:21
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