+
Skip to content

Conversation

Malus-risus
Copy link

No description provided.

Copy link
Member

@Him188 Him188 left a comment

Choose a reason for hiding this comment

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

如非必要, 我们一般不升级诸如 Koin, Gradle, AGP, media3, JNA 等依赖.

项目依赖太多了, 升级依赖也是风险.

每个升级依赖都是要看它的 changelog 的

@Malus-risus Malus-risus requested a review from Him188 October 1, 2025 15:20
@Him188
Copy link
Member

Him188 commented Oct 3, 2025

我的意思其实是我们不接受大批量升级依赖的 PR(

升级依赖需要我去读他们的 release notes 以及全面的测试

@Malus-risus
Copy link
Author

我的意思其实是我们不接受大批量升级依赖的 PR(

升级依赖需要我去读他们的 release notes 以及全面的测试

好的,我认为你可以慢慢审核过相关的log来判断是否需要升级。

@Malus-risus
Copy link
Author

我相信这需要很长的时间,如果不合适我可以关闭pull

@Him188
Copy link
Member

Him188 commented Oct 3, 2025

依赖确实是要升的, 一般驱动我升级依赖的是 material 3 的新组件 / bugfix, 以及 Kotlin 新功能. 升的时候会顺手看一眼其他的依赖.

@StageGuard
Copy link
Member

一个 PR 只能做一件事情,可以考虑去掉不必要的依赖更新,只支持 predictive back

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
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载