Replies: 1 comment
-
Hi! I think the alphabetical sort was there just to avoid the random ordering between runs, so it's no longer useful indeed You can open a PR to remove this 👍 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
I just realized that my 1 year old PR #293 cannot work (and never actually worked) because of the
sort
at the end of this line :metrics/source/plugins/notable/index.mjs
Lines 31 to 32 in ead26f7
Is it intended to sort alphabetically ?
If not, I can make the PR to fix this and sort by number of ⭐ , as planned originally in my PR.
Thanks for your answer
Beta Was this translation helpful? Give feedback.
All reactions