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

Handle glob pub use imports #34

Closed
obi1kenobi opened this issue Aug 7, 2022 · 1 comment
Closed

Handle glob pub use imports #34

obi1kenobi opened this issue Aug 7, 2022 · 1 comment
Labels
A-lint Area: new or existing lint C-enhancement Category: raise the bar on expectations E-help-wanted Call for participation: Help is requested to fix this issue. E-mentor Call for participation: Mentorship is available for this issue.

Comments

@obi1kenobi
Copy link
Owner

obi1kenobi commented Aug 7, 2022

Right now, glob imports are ignored for purposes of calculating the "importable paths" of an item.

This is likely to cause false-negative issues where items aren't scanned but they should be. If moving from a non-glob to a glob pub use, this could also cause a false-positive "item was removed" issue. The circumstances to trigger a false-positive are exceedingly rare, but this should be fixed soon regardless.

@epage epage added A-lint Area: new or existing lint C-enhancement Category: raise the bar on expectations E-help-wanted Call for participation: Help is requested to fix this issue. labels Aug 9, 2022
@obi1kenobi obi1kenobi added the E-mentor Call for participation: Mentorship is available for this issue. label Aug 10, 2022
@obi1kenobi
Copy link
Owner Author

Fixed by #327.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-lint Area: new or existing lint C-enhancement Category: raise the bar on expectations E-help-wanted Call for participation: Help is requested to fix this issue. E-mentor Call for participation: Mentorship is available for this issue.
Projects
None yet
Development

No branches or pull requests

2 participants