1959530 - Remove lint-baseline.xml for each Android component

Status: ASSIGNED

(Firefox for Android :: Tooling, task, P2)

Description

[Jonathan Almeida [:jonalmeida]]

We added a lint-baseline.xml for every component in order to land bug 1872229, however the consequences are that every new component now needs to generate a baseline file and then add it to the tooling/rewriting/Generated.txt file as well. This increases the friction developers will feel for wanting to add a new component.

The lint failures are not that many, so I think it's best to just fix them immediately so that we can remove the baseline files.

Comment 1

[Jonathan Almeida [:jonalmeida]]

I haven't gotten to this yet. Matt, is this something you'd be interested in as part of your lint baseline work?

Comment 2

[Matt Tighe [:matt-tighe]]

Thanks for the ping! This seems useful, I can add it to the code health backlog and forward it to the guild - perhaps someone there has capacity even though I don't currently. Without digging into the code, I'm wondering: will this affect new Gradle modules published in Fenix as well?

Comment 3

[Matt Tighe [:matt-tighe]]

Hey :jonalmeida, just didn't think to NI you last time but still curious to the answer of the question above.

Comment 4

[Jonathan Almeida [:jonalmeida]]

Yes, if we decided to take the path of adding gradle modules there, adding baseline files there will have to do the same thing we have for our existing gradle modules as well.