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
-
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