Fix lint warnings and reenable abortOnError about litho HOT 8 OPEN

@passy Is there a list of those, that you are ok with?

I mean there are lint errors like:

• view.setForeground(null); that are safe-guarded with an API check before accessing them, although Lint does not understand it. One solution could be to reverse the API check and run that method only then, that would satisfy the Lint I suppose.

from litho.

@pavlospt That sounds pretty reasonable.

We don't have a list right now. I think it might be best to tackle these in a few individual PRs. There may be some that are a bit more invasive and require some thought but a bunch of them are probably just missing annotations or places where we could use @SuppressLint if it's clear that Lint is overly sensitive.

from litho.

@passy Ok I understand what you mean. Will try to create a PR covering most of them so that we can discuss what is acceptable and should be suppressed and what not :)

from litho.

@pavlospt Awesome!

from litho.

If this one is still relevant I think I can prepare the PR I said some time ago 😂

from litho.

Oh yeah, that would be awesome. Would love to get rid of some more lint warnings. :)

from litho.

A nice alternative to this would be using a Lint baseline file and fix issues in parallel!

from litho.

Hey @pavlospt ! Were there any progress about that? I think we may enable Lint baseline for this config and switch abortOnError on again

from litho.