-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
feat: align some methods with the Change By Copy proposal #9207
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Ignored Deployments
|
bf7b646
to
71725f1
Compare
Codecov Report
@@ Coverage Diff @@
## main #9207 +/- ##
==========================================
+ Coverage 60.39% 60.41% +0.01%
==========================================
Files 233 233
Lines 16161 16169 +8
Branches 1232 1233 +1
==========================================
+ Hits 9760 9768 +8
Misses 6357 6357
Partials 44 44
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today! |
1b0706c
to
8005d6f
Compare
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://discord-js-git-fork-almeidx-collection-change-by-copy-discordjs.vercel.app/ |
BREAKING CHANGE: The `sorted` method has been renamed to `toSorted`
8005d6f
to
a53403b
Compare
Signed-off-by: RedGuy12 <paul@reid-family.org>
Please describe the changes this PR makes and why it should be merged:
Stage 4 proposal:
Overview of the changes:
sorted
->toSorted
toReversed
Status and versioning classification: