From 26f5286f97d53d2b30359234546eedc7d033e131 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Daniel=20Mart=C3=AD?= <mvdan@mvdan.cc>
Date: Tue, 21 Jun 2016 13:26:56 +0100
Subject: [PATCH] lint: don't unnecessarily disable warnings

Those that are sometimes false positives but could still point out valid
issues should be warnings, not disabled entirely.

The first two are warnings already, the third is an error.
---
 app/build.gradle | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/app/build.gradle b/app/build.gradle
index 63535c85d..9bb52526f 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -194,15 +194,8 @@ android {
         // Our translations are crowd-sourced
         disable 'MissingTranslation'
 
-        // We have locale folders like "values-he" and "values-id" as symlinks
-        // since some devices ship deprecated locale codes
-        disable 'LocaleFolder'
-
-        // Like supportsRtl or parentActivityName. They are on purpose.
-        disable 'UnusedAttribute'
-
         // to make CI fail on errors until this is fixed https://github.com/rtyley/spongycastle/issues/7
-        disable 'InvalidPackage'
+        warning 'InvalidPackage'
     }
 
     packagingOptions {