Skip to content

Conversation

@hannesa2
Copy link
Contributor

@hannesa2 hannesa2 commented Feb 9, 2019

It's related to #2441, most of the lints are now removed.
image

eg:
image

Still open :

  1. GradleCompatible: Incompatible Gradle Versions will be solved with AndroidX
  2. WrongConstant: Incorrect constant: no clue whats right here

@davigonz davigonz added this to the 2.11.0 milestone Feb 11, 2019
@davigonz davigonz requested a review from hosy February 13, 2019 11:14
@hannesa2 hannesa2 force-pushed the RemoveLintErrors branch 3 times, most recently from b85f08e to bf5ab94 Compare February 16, 2019 21:18
@davigonz davigonz self-requested a review February 26, 2019 17:04
@davigonz
Copy link
Contributor

davigonz commented Feb 26, 2019

@hannesa2 the code looks good to me but Android Studio Lint looks a bit different to your screenshot and it still shows some errors. Are you using a custom Lint o something like that?

@hannesa2
Copy link
Contributor Author

This lint has often a delay. Please do a clean/build and wait till lint scan all files in background. This background has sometime a delay

@hannesa2 hannesa2 force-pushed the RemoveLintErrors branch 2 times, most recently from 1bf0603 to b581181 Compare February 28, 2019 19:43
@hannesa2 hannesa2 force-pushed the RemoveLintErrors branch 2 times, most recently from 5ff9a3c to 204ace4 Compare March 9, 2019 06:25
@jesmrec jesmrec self-requested a review March 13, 2019 08:54
@jesmrec
Copy link
Collaborator

jesmrec commented Mar 13, 2019

Approved.

@hannesa2, you know how to do now

@hannesa2
Copy link
Contributor Author

done

@jesmrec
Copy link
Collaborator

jesmrec commented Mar 14, 2019

Transifex pushed before... we need another try.

@hannesa2
Copy link
Contributor Author

done

But the second request I don't found. But I rebased the second too

image

@jesmrec jesmrec merged commit 115d0ba into owncloud:master Mar 14, 2019
@jesmrec jesmrec mentioned this pull request Mar 14, 2019
@hannesa2 hannesa2 deleted the RemoveLintErrors branch March 14, 2019 08:16
@jesmrec jesmrec removed the Sprint label Mar 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants