Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Bump compileSdkVersion to 31 #4432

Merged
merged 3 commits into from
Oct 16, 2021
Merged

Bump compileSdkVersion to 31 #4432

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
1aeb254
Bump compileSdkVersion to 31
Oct 13, 2021
0fabc3e
Remove warning as error
Oct 14, 2021
f3901b6
Comment
Oct 15, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove warning as error
  • Loading branch information
Emmanuel Garcia committed Oct 14, 2021
commit 0fabc3e8bf0f0675ea663332117f5e904ffd592e
2 changes: 1 addition & 1 deletion packages/camera/camera/android/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
group 'io.flutter.plugins.camera'
version '1.0-SNAPSHOT'
def args = ["-Xlint:deprecation","-Xlint:unchecked","-Werror"]
Copy link
Contributor

@stuartmorgan-g stuartmorgan-g Oct 15, 2021

Choose a reason for hiding this comment

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

I assume the issue here is that the set of warnings will vary based on the build settings of the app depending on the plugin, so we don't want them to be errors?

Copy link
Author

@blasten blasten Oct 15, 2021

Choose a reason for hiding this comment

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

right. This is temporary as the fix is pending: #4423

def args = ["-Xlint:deprecation","-Xlint:unchecked"]
Copy link
Contributor

@mvanbeusekom mvanbeusekom Oct 14, 2021

Choose a reason for hiding this comment

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

Would the removal of the -Werror flag be permanent? If not maybe add a comment explaining why it is temporary removed?

Copy link
Author

@blasten blasten Oct 15, 2021

Choose a reason for hiding this comment

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

this is temporary and can be reverted after #4423


buildscript {
repositories {
Expand Down