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

Revert "Report used import when parent reference is already present (#437)" #532

Merged
merged 2 commits into from Jul 17, 2019

Conversation

shashachu
Copy link
Contributor

@shashachu shashachu commented Jul 17, 2019

This reverts commit 238b774.

Fixes #531, re-introduces #405

@sowmyav24 FYI I'm reverting this diff because it seems to have caused a couple regressions. I'd prefer the false-negative of the original issue the diff was fixing to the false positive of the regression.

@sowmyav24
Copy link
Contributor

@shashachu Sure will try to them to to the test cases

@shashachu shashachu merged commit b548fef into pinterest:master Jul 17, 2019
sowmyav24 pushed a commit to sowmyav24/ktlint that referenced this pull request Jul 18, 2019
…interest#437)" (pinterest#532)

* Revert "Report used import when parent reference is already present (pinterest#437)"

This reverts commit 238b774.

Fixes pinterest#437, re-introduces pinterest#405

* fix build
@shashachu shashachu deleted the revert-parent-imports branch August 2, 2019 22:16
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.

Extension imports from 3rd party library detected as "Unused import"
2 participants