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

Remove unnecessary wildcard imports #1258

Merged

Conversation

paul-dingemans
Copy link
Collaborator

Closes #1256

Description

Renamed and extended function removeBackticks to removeBackticksAndWildcars to remove the wildcard from the import path.

Checklist

  • tests are added
  • CHANGELOG.md is updated

@YauheniPo
Copy link

@Tapchicoma please review and merge this fix

@romtsn romtsn merged commit 3063316 into pinterest:master Oct 27, 2021
@YauheniPo
Copy link

@romtsn when will the fix be available in the new version of the plugin?

@romtsn
Copy link
Collaborator

romtsn commented Oct 27, 2021

There's no plugin in this repo... I don't know when it will be available in ktlint, I am not responsible for the release process, but rather @shashachu.

You could use a snapshot build in the meantime

@paul-dingemans
Copy link
Collaborator Author

Maybe the unofficial ktlint plugin for IntelliJ is meant. For this the owner of the plugin needs to be contacted as soon ktlint's new version is released.

@paul-dingemans paul-dingemans deleted the 1256-unnecessary-wildcard-imports branch October 30, 2021 10:11
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.

Unused imports are not removed with .* in the end
3 participants