-
Notifications
You must be signed in to change notification settings - Fork 503
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
Another differences between AndroidStudio and Ktlint formatting #796
Comments
Not sure if I should be dropping this here or in a separate issue, imports ordering is also off: Android Studio
vs ktlint 0.37.0
|
@jai are you running ktlint with |
thanks @romtsn so for happy coexistence between ktlint and Android Studio I'm going to have to disable the |
Either that or you could add |
@romtsn so I created a
Running ktlint ended up throwing a bunch of errors:
I'll try and dig deeper to see what's going on. This is ktlint running via a Gradle plugin so will try the command line ktlint and see if that makes a difference. |
This might be a bug (also please try to use 0.37.2 as there were some fixes for the import-ordering rule). If this still isn't working, feel free to open a new issue with code sample |
Looks like still having issues even with 0.37.2. Opened #826 |
The fix on the issue I've posted here #685 indeed fixed a lot of inconsistencies between formatting via ktlint and AndroidStudio, however, I found another bug.
This is the code before formatting, (ktlint sees no issues)
![Screenshot 2020-06-24 at 10 54 20](https://user-images.githubusercontent.com/29818981/85525953-ddc72c00-b609-11ea-8b55-850368094929.png)
After formatting via Android Studio
![Screenshot 2020-06-24 at 10 54 29](https://user-images.githubusercontent.com/29818981/85525947-dacc3b80-b609-11ea-96a9-ec8d2885e18c.png)
ktlint version 0.37.2
The text was updated successfully, but these errors were encountered: