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

Extensions should be sorted case insensitive #18808

Closed
davsclaus opened this issue Jul 19, 2021 · 5 comments · Fixed by #18815
Closed

Extensions should be sorted case insensitive #18808

davsclaus opened this issue Jul 19, 2021 · 5 comments · Fixed by #18815
Assignees
Labels
area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins
Milestone

Comments

@davsclaus
Copy link
Contributor

Describe the bug
(Describe the problem clearly and concisely.)

Extensions are sorted with case sensitive, eg A..Za..z

Expected behavior
(Describe the expected behavior clearly and concisely.)

They should be sorted regardless of case

Actual behavior
(Describe the actual behavior clearly and concisely.)

To Reproduce
Steps to reproduce the behavior:

  1. https://code.quarkus.io/?extension-search=camel

Screenshots
(If applicable, add screenshots to help explain your problem.)

See the screenshot

Environment (please complete the following information):

  • Browser:
  • OS:

Additional context
(Add any other context about the problem here.)

@davsclaus
Copy link
Contributor Author

sort-extensions

@davsclaus
Copy link
Contributor Author

If the extensions are sorted regardless of case, then

Camel gPRC will be listed together with the other G extensions, and so on.

@ia3andy
Copy link
Contributor

ia3andy commented Jul 19, 2021

good catch!

@ia3andy ia3andy transferred this issue from quarkusio/code.quarkus.io Jul 19, 2021
@ia3andy
Copy link
Contributor

ia3andy commented Jul 19, 2021

@davsclaus would you provide a PR?

@gastaldi gastaldi added area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins and removed triage/needs-triage labels Jul 19, 2021
@gastaldi gastaldi self-assigned this Jul 19, 2021
gastaldi added a commit to gastaldi/quarkus that referenced this issue Jul 19, 2021
gastaldi added a commit to gastaldi/quarkus that referenced this issue Jul 19, 2021
gastaldi added a commit to gastaldi/quarkus that referenced this issue Jul 19, 2021
gastaldi added a commit to gastaldi/quarkus that referenced this issue Jul 19, 2021
@quarkus-bot quarkus-bot bot added this to the 2.2 - main milestone Jul 19, 2021
@gsmet gsmet modified the milestones: 2.2 - main, 2.1.0.Final Jul 21, 2021
gsmet pushed a commit to gsmet/quarkus that referenced this issue Jul 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants