search

jashkenas / underscore

JavaScript's utility _ belt
https://underscorejs.org
MIT License
27.29k stars 5.53k forks source link

Alphabetize exports in underscore.js #2989

Closed saaasaab closed 9 months ago

saaasaab commented 9 months ago

This is just a cleanup ticket to alphabetize the exports in the main underscore.js file, leaving
proto: null, at the top and

'default': _$1 at the bottom

Group 870

jgonggrijp commented 9 months ago

@saaasaab the underscore.js is generated by Rollup in a commit hook, so your changes would be destroyed on the next commit.

The order is determined by the content of modules/index.js. That order has been carefully chosen in order to make the single read annotated source as readable as possible.

Why would you prefer the exports to be sorted alphabetically?

saaasaab commented 9 months ago

Thank you for explaining that. The reason I thought it was good for the exports to be alphabetical is mainly so it is easier to find what you're looking for when going through the code. Again, thank you for going though over the process.