Skip to content

Conversation

@HatakeKakashri
Copy link
Contributor

What is it?

  • Bugfix (user facing)
  • Feature (user facing) ⚠️ Your PR must target the refactor branch
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

  • This is required for using the Navigation3 library in Jetpack Compose, and the same was also discussed offline in the matrix room.
  • Updated the minSdk in app/build.gradle.kts.
  • Adjusted the api-level for the test-android min sdk CI workflow in .github/workflows/ci.yml.
  • Simplified the getOsString() method in ErrorActivity.java by removing the conditional check for Build.VERSION_CODES.M, as API 23 is now the minimum.

Before/After Screenshots/Screen Record

  • Before:
  • After:

Fixes the following issue(s)

  • Fixes #

APK testing

The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR. You can find more info and a video demonstration on this wiki page.

Due diligence

- Updated the `minSdk` in `app/build.gradle.kts`.
- Adjusted the api-level for the test-android min sdk CI workflow
  in `.github/workflows/ci.yml`.
- Simplified the `getOsString()` method in `ErrorActivity.java` by removing
  the conditional check for `Build.VERSION_CODES.M`, as API 23 is now the minimum.
@github-actions github-actions bot added the size/small PRs with less than 50 changed lines label Jan 13, 2026
@ale5000-git
Copy link

NewPipe is the most useful app on old devices.
Why block them?

@absurdlylongusername
Copy link
Member

@ale5000-git Basically Google is a cursed company that deprecates stuff and if we want to not be stuck on old libraries and be able to use newer features and stuff, then we need to increase minSdk otherwise development will be a lot harder for devs and available functionality will be limited.

Copy link
Member

@theimpulson theimpulson left a comment

Choose a reason for hiding this comment

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

Please remove all checks for Build.VERSION_CODES.M

@theimpulson theimpulson merged commit 61c25d4 into TeamNewPipe:refactor Jan 21, 2026
5 checks passed
@theimpulson
Copy link
Member

Thank you

@HatakeKakashri HatakeKakashri deleted the bump_min_sdk_23 branch January 21, 2026 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/small PRs with less than 50 changed lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants