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

Version Catalog の導入 #123

Merged
merged 5 commits into from
Oct 13, 2023
Merged

Conversation

yugg003
Copy link
Contributor

@yugg003 yugg003 commented Oct 11, 2023

解決したいこと

  • 依存しているライブラリの管理を Gradle 公式機能で行いたい

やったこと

  • Version Catalog を導入しました。
    • 直近でアップデート予定の依存関係・設定のみ移行しています
    • Gradle Plugin 関連は、plugins ブロックでかける形になるように id を変更しています。

やらないこと

スクリーンショット

UIに変更が生じた場合はスクショを貼ってください

Before After

動作確認環境

@@ -40,7 +40,6 @@ dependencies {
implementation deps.androidX.fragment
implementation deps.androidX.material
implementation deps.androidX.core
implementation deps.kotlin.stdlib
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Standard Library の依存関係は、実は細かく指定する必要がないため、この機会に削除しました。
https://kotlinlang.org/docs/gradle-configure-project.html#dependency-on-the-standard-library

@yugg003 yugg003 marked this pull request as ready for review October 11, 2023 06:41
android-application = { id = "com.android.application", version.ref = "android-gradle" }
android-library = { id = "com.android.library", version.ref = "android-gradle" }
kotlin = { id = "org.jetbrains.kotlin.android", version.ref = "kotlin" }
kotlin-parcelize = { id = "org.jetbrains.kotlin.plugin.parcelize", version.ref = "kotlin" }
Copy link
Contributor Author

Choose a reason for hiding this comment

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

id はここを参考に設定しています。
https://plugins.gradle.org/plugin/org.jetbrains.kotlin.plugin.parcelize

@verno3632 verno3632 self-requested a review October 11, 2023 11:03
@verno3632
Copy link
Contributor

LGTM

@verno3632 verno3632 merged commit cd47a13 into pixiv:main Oct 13, 2023
@yugg003 yugg003 deleted the feature/version_catalog branch March 12, 2024 09:05
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.

2 participants