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.

Sign up for GitHub

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

Jump to bottom

Report unused transparent inline imports #17252

Closed
szymon-rd opened this issue Apr 13, 2023 · 0 comments · Fixed by #20894
Closed

Report unused transparent inline imports #17252

szymon-rd opened this issue Apr 13, 2023 · 0 comments · Fixed by #20894
Labels
area:inline area:linting Linting warnings enabled with -W or -Xlint itype:bug

Comments

@szymon-rd
Copy link
Contributor

Compiler version

3.3.1-RC1-bin-20230411-d577300-NIGHTLY

Problem

After merging #17061, transparent inline imports are never reported as unused. However, we can use the information in derived trees to detect unused imports.
@szymon-rd szymon-rd added itype:bug stat:needs triage Every issue needs to have an "area" and "itype" label labels Apr 13, 2023
@szymon-rd szymon-rd self-assigned this Apr 13, 2023
@nicolasstucki nicolasstucki added area:inline area:linting Linting warnings enabled with -W or -Xlint and removed stat:needs triage Every issue needs to have an "area" and "itype" label labels Apr 17, 2023
@mbovel mbovel added the Spree Suitable for a future Spree label Feb 19, 2024
@mbovel mbovel removed the Spree Suitable for a future Spree label Sep 23, 2024
@som-snytt som-snytt assigned som-snytt and unassigned szymon-rd Nov 5, 2024
@som-snytt som-snytt mentioned this issue Jan 2, 2025
Merged
@som-snytt som-snytt removed their assignment Jan 21, 2025
@sjrd sjrd closed this as completed in 9cb97ec Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:inline area:linting Linting warnings enabled with -W or -Xlint itype:bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

In selector check, prefix of reference must match import qualifier som-snytt/dotty
4 participants
@som-snytt @mbovel @nicolasstucki @szymon-rd