appium / appium-espresso-driver

Espresso integration for Appium
Apache License 2.0
194 stars 75 forks source link

Kotlin Conversion #373

Open dpgraham opened 5 years ago

dpgraham commented 5 years ago

Because Kotlin is fully interoperable with Java, and .kt and .java files can mix-and-match, we can do this conversion piece-by-piece instead of just one big, messy refactor.

Here's my recommended ordering

Each step of the way, convert one file at a time and always check that the JUnit tests pass. Do any refactoring that fixes any broken tests. Run the e2e tests too. This will expose any other broken code.

Since, as said above, we can do it piece-by-piece, we can do several pull requests and don't need to do it in just one.

mykola-mokhnach commented 5 years ago

I'd rather convert from top to bottom. Handlers first, as they don't have any dependencies

dpgraham commented 5 years ago

Actually yeah, Handlers can be moved above Models