-
Notifications
You must be signed in to change notification settings - Fork 512
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
Backticks in imports should be ignored for ordering purposes #1106
Comments
Probably rarer, but it would also be possible for the escapes to appear as part of the package to.
|
Kotlin Plugin for IntelliJ still provides old imports ordering behavior. |
I have submitted a change to the JetBrains Kotlin plugin to match this ordering, and it will eventually be released in an update 😁 |
For reference: https://youtrack.jetbrains.com/issue/KTIJ-19394 According to YouTrack, should be part of 2022.1. |
Do I understand correctly that you have created inconsistency between IntelliJ platform and ktlint that will last at least a year before Android Studio gets stable on 2022.1 platform? And without any possibility to configure this? Is there a chance that some AS will get it sooner? Alternatively, would you be open to accept PR allowing to revert this with some configuration? |
Can confirm that we've hit this issue as well on Android as @hrach pointed out. It's indeed a bit weird that this change was initially done in June 2021, with support inside IDEA not even out yet (coming in roughly a full year after the initial change in KtLint) and with AS getting the update based off of the new IDEA version probably after 1 more year, in 2023. So almost 2 years difference in time between the 2 dependencies (KtLint and actual IDE)... Are there any contingencies in place for de-syncing issues like this? Can this be mitigated somehow for both the current situation as well as any possible future cases? |
Expected Behavior
Imports are ordered alphabetically regardless of quotes
Observed Behavior
Imports with backticks are sorted above any other imports at the same level
Steps to Reproduce
(See code samples above)
Your Environment
The text was updated successfully, but these errors were encountered: