Migrate Lint Rules to UAST #363

hotchemi · 2017-09-09T15:41:45Z

Issue

Migrate Lint Rules to use UAST from PSI
- To work with both Java/Kotlin! Check the forum post to get more detail
- Currently there's no document about that except Tor's post on group 😿
Fix test cases not to use old LintDetectorTest which is JUnit3 based one.
Update gradle to 4.1, android-gradle-plugin to 3.0.0-beta5, Kotlin to 1.1.3-2
Fix lint checks

hotchemi · 2017-09-11T02:10:18Z

@permissions-dispatcher/committers please review 🙇

hotchemi · 2017-09-11T08:30:01Z

Let me merge this PR since it includes some significant changes to proceed other task 🙇
If there's weird points let me know 🙇

mannodermaus · 2017-09-11T09:45:46Z

I was only able to read over the changes superficially, but everything looks good to me from here. Thank you for the hard work!

hotchemi added 2 commits September 9, 2017 20:10

Update Gradle, android-lint API, android gradle plugin ver.

52ed0d6

Implement with UAST API.

9e1ebbf

This was referenced Sep 9, 2017

3.0.0 Preparations #358

Merged

Migrate Lint Rules to UAST #359

Closed

hotchemi added 3 commits September 10, 2017 15:45

Tweak a few points.

d6f6773

Replace unit tests to new API, still failing though...

ab6c60a

Fix failed tests and polish a bit.

aab592e

hotchemi changed the title ~~[WIP] Migration Lint API to UAST from PSI~~ Migrate Lint Rules to UAST Sep 10, 2017

Fix lint errors.

fdcf932

hotchemi added 2 commits September 11, 2017 16:17

Add error message assert to each test cases.

47d23e1

Fix a logic of ONRequestPermissionsResultDetector.

6d6627b

hotchemi merged commit 9e78d03 into master Sep 11, 2017

hotchemi deleted the lint-uast branch September 11, 2017 08:30