Closed Bug 1252557 Opened 4 years ago Closed 4 years ago

Fix remaining lint issues so we can add lint to automation

Categories

(Firefox for Android :: General, defect)

All
Android
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 47
Tracking Status
firefox47 --- fixed

People

(Reporter: mcomella, Assigned: mcomella)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

There appear to be 4 remaining.
afaict, we don't currently support app indexing (bug 1252569) so this should be
okay to reduce the warning level. We can ignore it if we decide we don't want
to support this feature.

Review commit: https://reviewboard.mozilla.org/r/37431/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/37431/
Attachment #8725352 - Flags: review?(nalexander)
Comment on attachment 8725351 [details]
MozReview Request: Bug 1252557 - Add ignore for HardcodedDebugMode. r=nalexander

https://reviewboard.mozilla.org/r/37429/#review33987

Sure.
Attachment #8725351 - Flags: review?(nalexander) → review+
Comment on attachment 8725352 [details]
MozReview Request: Bug 1252557 - Add warning for GoogleAppIndexingDeepLinkError. r=nalexander

https://reviewboard.mozilla.org/r/37431/#review33989
Attachment #8725352 - Flags: review?(nalexander) → review+
Comment on attachment 8725353 [details]
MozReview Request: Bug 1252557 - Suppress ParcelCreator in AndroidFxAccount. r=nalexander

https://reviewboard.mozilla.org/r/37433/#review33991
Attachment #8725353 - Flags: review?(nalexander) → review+
You need to log in before you can comment on or make changes to this bug.