Skip to content

Conversation

@kix
Copy link
Contributor

@kix kix commented Jan 21, 2016

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets none
License MIT
Doc PR none

This makes the behaviour of config commands a bit more user-friendly. Scanning through an ordered list should be a lot easier for the user, especially if she knows what's the exact bundle they're looking for. The sort is applied to the bundle name, as the extension alias might not always be present.

Before:
before
After:
after

@fabpot
Copy link
Member

fabpot commented Jan 22, 2016

👍 I would merge it into some 2.x versions instead of master though.

@javiereguiluz
Copy link
Member

👍 nice improvement. Thanks @kix!

@linaori
Copy link
Contributor

linaori commented Jan 22, 2016

This will save you a lot of eye peering in bigger projects 👍

@kix
Copy link
Contributor Author

kix commented Jan 22, 2016

OK, will rebase and reopen against 2.7/2.8 as the latest current versions. Right @fabpot?

@sstok
Copy link
Contributor

sstok commented Jan 22, 2016

👍

@kix kix closed this Jan 23, 2016
@kix
Copy link
Contributor Author

kix commented Jan 23, 2016

Merged as #17495 instead

@kix kix deleted the sort-bundles-in-configdump branch January 23, 2016 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants