-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Use NotificationChannelCompat. #6633
Use NotificationChannelCompat. #6633
Conversation
AppCompat 1.2.0 to 1.3.0 is a very important library upgrade, being one of the most used core libraries of NewPipe. I'm of the opinion that this be thoroughly tested before being merged, at least to catch possible new behaviors and bugs before new GitHub issues are created. AppCompat 1.3.0 changelog: https://developer.android.com/jetpack/androidx/releases/appcompat#version_130_3 Edit: compileSdk should not be bumped here either. |
6503da4
to
bfc2c36
Compare
Thank you! I would like to merge all your recent prs, so also #6634, #6638, #6675, #6685 apart from #6675 all PRs do increase the appcompat version. Could you do that in another PR seperately and rebase your PRs accordingly? Also do you mind if we condense your PRs in the changelog like "general codequality updates" linking to your PRs? |
Yeah, that's fine. |
I was probablly a bit too quick in merging the appcompat lib update as I wanted to get that our of way to be able to merge these PRs. The transitive dependencies are not matching. I guess we need to adopt. |
Would be a good idea, yes 😉 |
bfc2c36
to
a4c207f
Compare
a4c207f
to
d324502
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! It works, both on API 19 emulated, API 29 real and API 30 emulated
What is it?
Description of the changes in your PR
NotificationChannelCompat
.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.
Due diligence