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

Sort completion items #506

Merged
merged 7 commits into from
Jul 7, 2022
Merged

Sort completion items #506

merged 7 commits into from
Jul 7, 2022

Conversation

ryuukk
Copy link
Contributor

@ryuukk ryuukk commented Jun 22, 2022

It makes it easier to choose and find what you exactly want, things are properly categorized, then sorted a-z

image

image

@SuperAuguste
Copy link
Member

Tests fail; could you please update them by adding sortText to the expected values? Thanks! :)

@ryuukk
Copy link
Contributor Author

ryuukk commented Jun 24, 2022

Done!

@SuperAuguste
Copy link
Member

Thank you so much! I'll let the CI run and then merge!

@SuperAuguste
Copy link
Member

Please update all the completion tests accordingly; sorry for the hassle but it's important! :)

@SuperAuguste SuperAuguste merged commit 7fa58ae into zigtools:master Jul 7, 2022
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 this pull request may close these issues.

None yet

2 participants