-
Notifications
You must be signed in to change notification settings - Fork 14
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
An option to sort sections #93
Conversation
Awesome! Thanks 😃 I'll have a closer look tomorrow. But yes, I agree that a simple |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey @funkyfuture! thanks a lot for the input here! The changes look great! I only have two minor suggestions for the docs, but otherwise I am happy to merge this when you are done here and make a new release 👍
please let me know @funkyfuture if there is anything else from your side or if I can merge this PR as it is |
Co-authored-by: Philipp S. Sommer <philipp.sommer@hereon.de>
thanks for the feedback. i'm happy with the solution as it is. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome! 🥳
this is a solution for the proposal in #90.
from a user perspective, i'd actually rather prefer to simply provide aTrue
to achieve simple alphabetical sorting. shall i extend to this?should i add an example where a dict with weights to define the order is used?i also did some occasional modernization of code as one may consider it. are these changes okay?