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

Use List.of(). #4816

Closed

Conversation

Isira-Seneviratne
Copy link
Member

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

  • Update compileSdkVersion to 30.
  • Use the List.of() factory methods introduced in API level 30, which are desugared for API levels below 30:
    Screenshot 2020-11-06 101401

APK testing

debug.zip

Due diligence

@opusforlife2
Copy link
Collaborator

Was the build.gradle change intentional?

@Isira-Seneviratne
Copy link
Member Author

Was the build.gradle change intentional?

Yeah, because List.of() (and the other Java 9 collection factory methods) were added in API level 30.

Copy link
Member

@Stypox Stypox left a comment

Choose a reason for hiding this comment

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

@Isira-Seneviratne code looks good to me. Thank you!
Are you sure that increasing the compileSdkVersion to 30 doesn't break backwards-compatibility with API 19, since we're using the old file:// scheme there which afaik is not allowed on Android 11?
See #3777

@TobiGr
Copy link
Contributor

TobiGr commented Nov 20, 2020

We should not touch the minSDKVersion or the compileSDKVersion until F-Droid fixed their v2 signing for reproducible builds.

@Isira-Seneviratne
Copy link
Member Author

@Isira-Seneviratne code looks good to me. Thank you!
Are you sure that increasing the compileSdkVersion to 30 doesn't break backwards-compatibility with API 19, since we're using the old file:// scheme there which afaik is not allowed on Android 11?
See #3777

I didn't know there was such a restriction.

@Isira-Seneviratne Isira-Seneviratne force-pushed the Use_List_of branch 2 times, most recently from a648a16 to 696b023 Compare November 29, 2020 02:15
@Isira-Seneviratne Isira-Seneviratne force-pushed the Use_List_of branch 3 times, most recently from 93a017d to 88b5440 Compare December 19, 2020 23:07
@Isira-Seneviratne Isira-Seneviratne force-pushed the Use_List_of branch 2 times, most recently from 24d5e04 to 6d241e2 Compare December 27, 2020 03:53
@Isira-Seneviratne Isira-Seneviratne force-pushed the Use_List_of branch 2 times, most recently from 326469e to 261795d Compare January 18, 2021 00:51
@TobiGr TobiGr force-pushed the dev branch 2 times, most recently from 679bc75 to 2aeccc0 Compare March 16, 2021 08:24
@triallax triallax added the codequality Improvements to the codebase to improve the code quality label Jun 13, 2021
@litetex litetex marked this pull request as draft October 2, 2021 16:33
@litetex
Copy link
Member

litetex commented Oct 2, 2021

Closing this for now:

  • no progress
  • no GitHub actions builds (Tests are not executed!)
  • merge conflicts

Also in my opinion this PR should be split into 2 smaller parts:

  1. Where the compileSDK is updated
  2. Where the code improvements are done

Makes it easier and faster to review.

Feel free to reopen it when there is progress again 😄

@litetex litetex closed this Oct 2, 2021
@litetex litetex mentioned this pull request Oct 10, 2021
2 tasks
@Isira-Seneviratne Isira-Seneviratne deleted the Use_List_of branch July 15, 2022 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
codequality Improvements to the codebase to improve the code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants