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

Add Android example to gradle plugin readme. #168

Merged
merged 2 commits into from
Dec 1, 2017

Conversation

runningcode
Copy link
Contributor

Ref #111

@@ -134,6 +134,19 @@ spotless {
}
```

## Applying to Android Java source
Be sure to add `target "**/*.java` otherwise spotless will not detect Java code inside Android modules.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Missing a trailing "
  • Everywhere else in the readme we use ' instead of "
  • Let's make this a ### For Android Java after ## Applying to Java source, right before ## Applying to Java source (google-java-format)

Thanks for the PR, this is a good idea

@JLLeitschuh
Copy link
Member

JLLeitschuh commented Nov 27, 2017

The same thing I think is true for Kotlin as well. Due to how flavors work. Maybe add a note about that as well?

@runningcode
Copy link
Contributor Author

@nedtwigg Lines fixed!
@JLLeitschuh You're probably right, but I haven't used diffplug with kotlin and I don't want to write any false information, but you're welcome to open a PR with those suggestions.

@nedtwigg nedtwigg merged commit 34edc24 into diffplug:master Dec 1, 2017
@nedtwigg
Copy link
Member

nedtwigg commented Dec 1, 2017

Thanks @runningcode!

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.

3 participants