search

nwolverson / purescript-language-server

MIT License
184 stars 41 forks source link

Prioritize "Organise Imports" action lower than others #113

Closed andys8 closed 3 years ago

andys8 commented 3 years ago

Issue

Having a non imported function, code actions can be used to import it. With recent changes "organize imports" is at the top of the list

image

Proposed Solution

Prioritize "Organize Imports" lower regarding the list position. In the example "Import spy (Debug.Trace)" is what the user is likely gonna do.

nwolverson commented 3 years ago

I assume this is just based on the ordering of the actions, can easily change that. FYI this action doesn't appear in vscode, because it filters out source.organizeImports code action kind to a separate menu. No idea what the "correct" behaviour might be as it seems to be unspecified

andys8 commented 3 years ago

Would moving "Organise imports" to the end of the list be fine for you?

nwolverson commented 3 years ago

Yep, that's the plan - just not been pushing code recently.