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

Ordering the phpactor quickfix suggestions by relevance #2820

Open
mamazu opened this issue Jan 9, 2025 · 1 comment · May be fixed by #2840
Open

Ordering the phpactor quickfix suggestions by relevance #2820

mamazu opened this issue Jan 9, 2025 · 1 comment · May be fixed by #2840

Comments

@mamazu
Copy link
Contributor

mamazu commented Jan 9, 2025

When I invoke the quick fix action on an unused import then this is the order the quick fix actions are suggested.

image

While the "Remove unused import" option is there it's not the default option. I think we should prioritize fix options that depend on the current token over generic ones that always pop up.

@mamazu mamazu linked a pull request Feb 8, 2025 that will close this issue
@mamazu
Copy link
Contributor Author

mamazu commented Feb 9, 2025

Alternatively maybe a usage based approach could also work, like the more often you select an option the higher it ranks in the list.

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 a pull request may close this issue.

1 participant