You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Yes! We need some collective thinking on what to do with this information. My first instinct is that if we're able to match an import up to any of the dependencies in that list, then we should just do so, and not notify the user about the others (at least not by default).
This is something to be handled in later PRs anyway. I just wanted to get the basic functionality into FD with this PR. In a future PR, it might be that we want to use the packages_distributions() mapping more directly and throw away find_import_names_from_package_name() altogether.
And how does importlib resolve the dependency when it domes to calling them? It has a fallback, when cannot find a function in the first dependency and then goes to second and so on? Definitely to be investigated.
The text was updated successfully, but these errors were encountered:
From the discussion on PR #139
@Nour-Mws:
@jherland:
@mknorps:
The text was updated successfully, but these errors were encountered: