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

MediaType.get(s) -> s.toMediaType() #41

Merged
merged 1 commit into from
May 9, 2024
Merged

Conversation

chromy
Copy link
Contributor

@chromy chromy commented May 9, 2024

OkHttp 4 moves MediaType.get to toMediaType (an extension function) See: https://square.github.io/okhttp/changelogs/upgrading_to_okhttp_4/

@rbro112: could we chat about this quickly at some point? I needed to tweak this to get hn to build recently but I'm not clear on where the version of okhttp is chosen.

@chromy chromy marked this pull request as draft May 9, 2024 21:52
OkHttp 4 moves MediaType.get to toMediaType (an extension function)
See: https://square.github.io/okhttp/changelogs/upgrading_to_okhttp_4/
@chromy chromy force-pushed the hjd/2024-05-09-mediaType branch from 4cec519 to 6f6612b Compare May 9, 2024 22:00
Copy link

emerge-tools bot commented May 9, 2024

2 builds increased size

Name Version Download Change Install Change Approval
Hacker News
com.emergetools.hackernews
1.0 (1) 2.0 MB ⬆️ 22.2 kB (1.14%) 4.6 MB ⬆️ 43.1 kB (0.95%) N/A
Hacker News
com.emergetools.hackernews.debug
1.0 (1) 17.7 MB ⬆️ 284.1 kB (1.63%) 60.2 MB ⬆️ 694.6 kB (1.17%) N/A

Hacker News 1.0 (1)
com.emergetools.hackernews

⚖️ Compare build
⏱️ Analyze build performance

Total install size change: ⬆️ 43.1 kB (0.95%)
Total download size change: ⬆️ 22.2 kB (1.14%)

Largest size changes

Item Install Size Change Download Size Change
🗑 kotlinx.coroutines.YieldKt ⬇️ -94.6 kB ⬇️ -43.2 kB
🗑 okio.Base64 ⬇️ -79.0 kB ⬇️ -36.1 kB
📝 kotlin.text.UStringsKt ⬆️ 73.7 kB ⬆️ 33.6 kB
📝 _COROUTINE._BOUNDARY ⬆️ 72.7 kB ⬆️ 33.1 kB
📝 kotlin.ResultKt ⬆️ 72.5 kB ⬆️ 33.1 kB
View Treemap

Image of diff

Hacker News 1.0 (1)
com.emergetools.hackernews.debug

⚖️ Compare build
⏱️ Analyze build performance

Total install size change: ⬆️ 694.6 kB (1.17%)
Total download size change: ⬆️ 284.1 kB (1.63%)

Largest size changes

Item Install Size Change Download Size Change
📝 okio.internal._BufferKt ⬆️ 25.5 kB ⬆️ 9.2 kB
publicsuffixes.gz ⬆️ 4.6 kB ⬆️ 4.6 kB
📝 okio.internal._ByteStringKt ⬆️ 12.6 kB ⬆️ 4.5 kB
okio.Buffer ⬆️ 8.9 kB ⬆️ 3.2 kB
📝 okio.internal._RealBufferedSourceKt ⬆️ 8.5 kB ⬆️ 3.1 kB
View Treemap

Image of diff


🛸 Powered by Emerge Tools

@chromy chromy marked this pull request as ready for review May 9, 2024 22:13
@chromy chromy merged commit 8c4937f into main May 9, 2024
4 of 5 checks passed
@chromy chromy deleted the hjd/2024-05-09-mediaType branch May 9, 2024 22:14
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