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.

Sign up for GitHub

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

Jump to bottom

[android] setup linter for autodispose usage #19

Closed
AttwellBrian opened this issue Oct 13, 2017 · 2 comments
Closed

[android] setup linter for autodispose usage #19

AttwellBrian opened this issue Oct 13, 2017 · 2 comments
Labels
help wanted
Projects
Android Initial release

Comments

@AttwellBrian
Copy link
Contributor

This is one of the things that makes RIBs so resilient to memory leaks. It is also a cool tech demo.

Could either hook up error prone. Or just write a quick psi linter as a tech demo.
@artman artman added this to Backlog in Android Oct 13, 2017
@artman artman added this to Backlog in Initial release Oct 13, 2017
@ZacSweers
Copy link
Contributor

UAST lint would be ideal since it would work for both kotlin and Java as well as show up in the IDE.

@AttwellBrian AttwellBrian moved this from Backlog to Done in Initial release Nov 2, 2017
@AttwellBrian AttwellBrian moved this from Done to Backlog in Initial release Nov 2, 2017
@AttwellBrian
Copy link
Contributor Author

I just landed the error-prone check instead.

@AttwellBrian AttwellBrian moved this from Backlog to Done in Initial release Nov 3, 2017
@AttwellBrian AttwellBrian moved this from Done to Backlog in Initial release Nov 3, 2017
@artman artman closed this as completed Mar 11, 2018
@artman artman moved this from Backlog to Done in Initial release Mar 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted
Projects
Initial release
Done
Android
Backlog
Milestone
No milestone
Development

No branches or pull requests
3 participants
@artman @AttwellBrian @ZacSweers