Skip to content

Comments

manager: remove Coil dependency and add OkHttp directly#3154

Merged
aviraxp merged 3 commits intotiann:mainfrom
u9521:dep-clean
Jan 14, 2026
Merged

manager: remove Coil dependency and add OkHttp directly#3154
aviraxp merged 3 commits intotiann:mainfrom
u9521:dep-clean

Conversation

@u9521
Copy link
Contributor

@u9521 u9521 commented Jan 14, 2026

Removed the Coil Compose dependency as it's no longer needed after migrating to MiuiX for loading app icons.

Added OkHttp dependency directly (previously it was a transitive dependency through Coil).

u9521 added 2 commits January 14, 2026 09:14
Remove redundant null checks and non-null assertions for OkHttp response bodies.

Signed-off-by: u9521 <[email protected]>
Signed-off-by: u9521 <[email protected]>
@aviraxp aviraxp merged commit 139b813 into tiann:main Jan 14, 2026
16 checks passed
@u9521 u9521 deleted the dep-clean branch January 14, 2026 10:37
PhuScam pushed a commit to PhuScam/KernelSU that referenced this pull request Feb 5, 2026
Removed the Coil Compose dependency as it's no longer needed after
migrating to MiuiX for loading app icons.


Added OkHttp dependency directly (previously it was a transitive
dependency through Coil).

---------

Signed-off-by: u9521 <[email protected]>
PhuScam pushed a commit to PhuScam/KernelSU that referenced this pull request Feb 18, 2026
Removed the Coil Compose dependency as it's no longer needed after
migrating to MiuiX for loading app icons.


Added OkHttp dependency directly (previously it was a transitive
dependency through Coil).

---------

Signed-off-by: u9521 <[email protected]>
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