Skip to content

Commit

Permalink
Suppress lint errors in Dialog module. remove abortOnError=false (fac…
Browse files Browse the repository at this point in the history
…ebook#19740)

Summary:
Suppress lint errors in Dialog module. remove abortOnError=false.

This might hide future problems, so removing it
``` gradle
lintOptions {
  abortOnError false
}
```

Builds locally just fine. But CI is failing for unknown reasons. https://circleci.com/gh/dulmandakh/react-native/265

RNTester will built without errors
Closes facebook#19740

Differential Revision: D8450600

Pulled By: hramos

fbshipit-source-id: faf508a0c546af18a05ee224628f88b02a38ab9f
  • Loading branch information
dulmandakh authored and macdoum1 committed Jun 28, 2018
1 parent 42f1a66 commit 91eed8f
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 12 deletions.
4 changes: 0 additions & 4 deletions RNTester/android/app/build.gradle
Expand Up @@ -121,10 +121,6 @@ android {
}
}

lintOptions {
abortOnError false
}

// applicationVariants are e.g. debug, release
applicationVariants.all { variant ->
variant.outputs.each { output ->
Expand Down
Expand Up @@ -9,6 +9,7 @@

import javax.annotation.Nullable;

import android.annotation.SuppressLint;
import android.app.AlertDialog;
import android.app.Dialog;
import android.app.DialogFragment;
Expand All @@ -34,6 +35,7 @@ public AlertFragment() {
mListener = null;
}

@SuppressLint("ValidFragment")
public AlertFragment(@Nullable DialogModule.AlertFragmentListener listener, Bundle arguments) {
mListener = listener;
setArguments(arguments);
Expand Down
Expand Up @@ -9,6 +9,7 @@

import javax.annotation.Nullable;

import android.annotation.SuppressLint;
import android.app.Dialog;
import android.content.DialogInterface;
import android.os.Bundle;
Expand All @@ -27,6 +28,7 @@ public SupportAlertFragment() {
mListener = null;
}

@SuppressLint("ValidFragment")
public SupportAlertFragment(@Nullable DialogModule.AlertFragmentListener listener, Bundle arguments) {
mListener = listener;
setArguments(arguments);
Expand Down
8 changes: 0 additions & 8 deletions build.gradle
Expand Up @@ -35,12 +35,4 @@ allprojects {
url "$androidSdk/extras/m2repository/"
}
}

project.configurations.all {
resolutionStrategy.eachDependency { details ->
if (details.requested.group == 'com.facebook.soloader') {
details.useVersion "0.3.0"
}
}
}
}

0 comments on commit 91eed8f

Please sign in to comment.