-
Notifications
You must be signed in to change notification settings - Fork 50
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
exception when using kotlin EAP #80
Comments
Thanks for the early warning. Looks like some definitions in the underlying compiler changed here: JetBrains/kotlin@efa59bf And of course Most pragmatic course of action is probably to wait for the next kotlin version to go official then upgrade ktlint/kotlinter to it with a compatibility note about which versions harmonize. |
Kotlin 1.3.20 is now officially out, so this issue has risen in urgency. |
Yes, indeed! I'm ready to take the update as soon as https://github.com/shyiko/ktlint is upgraded upstream. Perhaps someone would lend @shyiko a hand by contributing a PR? I might have a little time to contribute later in the week if nobody gets to it first. |
ktlin merged the fix: pinterest/ktlint@2fa9b74 can kotlinter use jitpack until there is a matching release ? |
I don't believe this project is compatible with jitpack: And it's probably good practice to give the maintainer an opportunity to verify any changes they committed since the last release and then tag it officially. |
ktlint 0.30.0 has been released: pinterest/ktlint@43a83d0 |
Fixes #80 and takes improvements in the latest ktlint
Fixes #80 and takes improvements in the latest ktlint
Thank you everyone! You should now find kotlinter 1.21.0 compatible with latest kotlin. |
using
Kotlin 1.3.20 EAP 2
or1.3.20-eap-52
it fails like thisalso reported here: pinterest/ktlint#321
The text was updated successfully, but these errors were encountered: