Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lint cleanup #902

Open
wants to merge 20 commits into
base: edge
Choose a base branch
from
Open

Lint cleanup #902

wants to merge 20 commits into from

Conversation

vrih
Copy link
Contributor

@vrih vrih commented Oct 21, 2018

Apologies for the size of this commit. This is bringing the code up to modern Java standards and is almost entirely implementing the recommendations from android-lint. I've tried to break this up into themed commits to make review a bit easier.

I don't think there's anything too controversial in there and I'm hoping cleaning this up will make it easier to move forward and tackle things like deprecated functions.

@vrih vrih force-pushed the lint-cleanup branch 2 times, most recently from 34e2cf1 to 182c95b Compare October 27, 2018 17:49
@vrih vrih force-pushed the lint-cleanup branch 2 times, most recently from 42881ea to 4273a05 Compare November 23, 2018 20:32
@antgel
Copy link

antgel commented Mar 27, 2021

A shame that this was never merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants