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

Sort commands under namespaces alphabetically #4055

Closed
bpstr opened this issue Apr 30, 2019 · 1 comment
Closed

Sort commands under namespaces alphabetically #4055

bpstr opened this issue Apr 30, 2019 · 1 comment

Comments

@bpstr
Copy link

bpstr commented Apr 30, 2019

Commands under namespaces are sorted in the same order as defined in files which makes hard to find the right commands. Especially in custom namespaces defined in multiple classes. Example:

 devel: 
   devel:reinstall (dre)  
   devel:hook (fnh, fn-hook, hook)   
   devel:event (fne, fn-event, event)  
   devel:token (token)
   devel:uuid (uuid)
   devel:services

I would like to see the commands in alphabetical order inside namespaces.

Another solution would be to add an optional argument to define sorting method under namespaces.

This is a small modification but could improve user experience and may be easy to solve.
@bpstr bpstr mentioned this issue Apr 30, 2019
Closed
@greg-1-anderson
Copy link
Member

🎉

m4olivei added a commit to m4olivei/drush that referenced this issue May 24, 2019
@m4olivei
Merge branch 'master' into issue_4032_queue-run-lease-time
afae951 
* master:
  Remove latest unstable chiclet. Its not helpful.
  Remove mentions of Shippable. (drush-ops#4079)
  Add mysql service to Travis config. (drush-ops#4078)
  Fix link in docs.
  More docs about dynamic users in site.yml files.
  Fix drush-ops#4055. Sort inside namespaces in list command. (drush-ops#4057)
  Print class name in drush_op()
  Update dependencies. (drush-ops#4051)
  UNISH_NO_TIMEOUTS is unused since we moved to CLITrait. (drush-ops#4047)
  Add jsonapi command (drush-ops#4045)
  Fixes drush-ops#4034: Ensure that status code is nonzero on invalid shutdown (drush-ops#4037)
  Add webflo/drupal-core-require-dev (drush-ops#4041)
  Fix TTY detection for runserver command. (drush-ops#4033)
  Add vagrant transport to Drush. (drush-ops#4036)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@greg-1-anderson @bpstr