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

[SR-7828] Add name-based fixits to product dependencies name #1747

Closed
wants to merge 1 commit into from

Conversation

JhonnyBillM
Copy link
Contributor

Added "did you mean ___ ?" when a product dependency cannot be found.

Please check my implementation with map and reduce, I am not sure if that is perfectly correct (in terms of readability, guidelines, etc).

@ankitspd
Copy link
Member

This will be very cool to add! We should make the implementation a bit smart to only suggest things when it makes sense. You can take a look at this method from compiler for some inspiration https://github.com/apple/swift/blob/876738fabe1218b824f26b665f8808c18c0a7513/lib/Sema/TypeCheckNameLookup.cpp#L579-L607

@ankitspd ankitspd added the WIP Work in progress label Aug 20, 2018
@ankitspd
Copy link
Member

Closing due to inactivity, feel free to update and reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants