From 25cff3d9b02c108ef79d67001a8b5721de4bffd4 Mon Sep 17 00:00:00 2001 From: Quentin Hibon Date: Mon, 7 Aug 2023 23:04:37 +0200 Subject: [PATCH 1/3] Check android lint violations against current baseline --- build.sh | 1 + uhabits-android/build.gradle.kts | 5 + uhabits-android/lint-baseline.xml | 7966 +++++++++++++++++++++++++++++ 3 files changed, 7972 insertions(+) create mode 100644 uhabits-android/lint-baseline.xml diff --git a/build.sh b/build.sh index ac81802e1..3a9937a58 100755 --- a/build.sh +++ b/build.sh @@ -64,6 +64,7 @@ fail() { core_build() { log_info "Building uhabits-core..." $GRADLE ktlintCheck || fail + $GRADLE lintDebug || fail $GRADLE :uhabits-core:build || fail } diff --git a/uhabits-android/build.gradle.kts b/uhabits-android/build.gradle.kts index 5bcfef429..d8f6b23b9 100644 --- a/uhabits-android/build.gradle.kts +++ b/uhabits-android/build.gradle.kts @@ -76,6 +76,11 @@ android { buildFeatures { viewBinding = true } + + lint { + baseline = file("lint-baseline.xml") + warningsAsErrors = true + } } dependencies { diff --git a/uhabits-android/lint-baseline.xml b/uhabits-android/lint-baseline.xml new file mode 100644 index 000000000..2607914ff --- /dev/null +++ b/uhabits-android/lint-baseline.xml @@ -0,0 +1,7966 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + From 093591fbafd7f547295776ff3a699851c614226f Mon Sep 17 00:00:00 2001 From: "Alinson S. Xavier" Date: Thu, 4 Apr 2024 21:24:09 -0500 Subject: [PATCH 2/3] Update lint baseline --- uhabits-android/lint-baseline.xml | 456 ++++++++++++++++++++++++------ 1 file changed, 371 insertions(+), 85 deletions(-) diff --git a/uhabits-android/lint-baseline.xml b/uhabits-android/lint-baseline.xml index 2607914ff..35b787313 100644 --- a/uhabits-android/lint-baseline.xml +++ b/uhabits-android/lint-baseline.xml @@ -1,5 +1,27 @@ - + + + + + + + + + @@ -30,7 +52,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -41,7 +63,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -52,7 +74,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -63,7 +85,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -123,13 +145,13 @@ + id="UnusedAttribute" + message="Attribute `localeConfig` is only used in API level 33 and higher (current min is 28)" + errorLine1=" android:localeConfig="@xml/locales_config"" + errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -140,7 +162,7 @@ errorLine2=" ~~~~~~~~~"> @@ -195,7 +217,7 @@ errorLine2=" ~~~~"> @@ -217,7 +239,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -245,57 +267,13 @@ - - - - - - - - + message="A newer version of androidx.test.uiautomator:uiautomator than 2.2.0 is available: 2.3.0" + errorLine1=" androidTestImplementation("androidx.test.uiautomator:uiautomator:2.2.0")" + errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> - - - - - - - - + column="32"/> + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -1252,7 +1538,7 @@ errorLine2=" ~~~~~"> @@ -1285,7 +1571,7 @@ errorLine2=" ~~~~~~~~~~~~~~~"> @@ -4152,7 +4438,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -4163,7 +4449,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~"> @@ -4174,7 +4460,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -4185,7 +4471,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -4196,7 +4482,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~"> @@ -4207,7 +4493,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -4853,13 +5139,13 @@ id="IconDipSize" message="The image `ic_launcher_monochrome.png` varies significantly in its density-independent (dip) size across the various density versions: mipmap-hdpi/ic_launcher_monochrome.png: 108x108 dp (162x162 px), mipmap-mdpi/ic_launcher_monochrome.png: 162x162 dp (162x162 px), mipmap-xhdpi/ic_launcher_monochrome.png: 108x108 dp (216x216 px), mipmap-xxhdpi/ic_launcher_monochrome.png: 108x108 dp (324x324 px), mipmap-xxxhdpi/ic_launcher_monochrome.png: 108x108 dp (432x432 px)"> + file="src/main/res/mipmap-xxhdpi/ic_launcher_monochrome.png"/> - + @@ -5110,7 +5396,7 @@ errorLine2=" ~~~~~~~~~~~~~"> @@ -5143,7 +5429,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~"> @@ -5205,7 +5491,7 @@ @@ -7767,7 +8053,7 @@ Date: Thu, 4 Apr 2024 21:36:51 -0500 Subject: [PATCH 3/3] lint: disable GradleDependency --- uhabits-android/build.gradle.kts | 1 + 1 file changed, 1 insertion(+) diff --git a/uhabits-android/build.gradle.kts b/uhabits-android/build.gradle.kts index 7dbd220d7..28154d587 100644 --- a/uhabits-android/build.gradle.kts +++ b/uhabits-android/build.gradle.kts @@ -94,6 +94,7 @@ android { lint { baseline = file("lint-baseline.xml") + disable += "GradleDependency" warningsAsErrors = true } }