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

Update README with Kotlin support #159

Merged
merged 3 commits into from
Nov 6, 2017
Merged

Update README with Kotlin support #159

merged 3 commits into from
Nov 6, 2017

Conversation

StephenVinouze
Copy link
Contributor

Addressing Issue #123 about Kotlin support. I documented how to use this library with this language, with a few tips about a few rough edges that I have bumped into while discovering how to configure your library with Kotlin ;)

Copy link
Contributor

@gabrieloshiro gabrieloshiro left a comment

Choose a reason for hiding this comment

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

Mispelling

README.md Outdated
@@ -239,6 +239,68 @@ Maven:
</dependency>
```

Kotlin
-----
For all Kotlin enthousiasts, you may wonder how to use this library to configure your intents. This is perfectly compatible, with a bit of understanding of how Kotlin works, especially when it comes to annotation processing.
Copy link
Contributor

Choose a reason for hiding this comment

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

  • enthusiasts

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done ;)

README.md Outdated
apply plugin: 'kotlin-kapt'

dependencies {
compile 'com.f2prateek.dart:henson:(insert latest version)'
Copy link
Contributor

Choose a reason for hiding this comment

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

you can replace with implementation as compile is deprecated

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done and ready for merge ;)

@stephanenicolas stephanenicolas merged commit d2c8200 into f2prateek:master Nov 6, 2017
@stephanenicolas
Copy link
Collaborator

Thx @StephenVinouze

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.

None yet

3 participants