Skip to content
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

Add new toSorted, toSpliced and toReversed methods to "Array methods" article #3622

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Hasan-Mir
Copy link

@Hasan-Mir Hasan-Mir commented Dec 8, 2023

This pull request introduces additional notes to the documentation of Array methods. The aim is to inform readers about new methods that perform similar operations to splice, sort, and reverse, but do not mutate the original array. These methods are toSpliced, toSorted, and toReversed.

@damonholden
Copy link

yeah lets get this updated. Also worth mentioning the .with() Array method aswell. Also some of the tasks like "copy and sort" in this section need to be updated to include the new apis.

@smith558 smith558 self-assigned this Feb 14, 2024
@smith558
Copy link
Member

Also worth mentioning the .with() Array method aswell. Also some of the tasks like "copy and sort" in this section need to be updated to include the new apis.

Would you be interested in opening a PR for this after this one gets merged (will be soon)? 🙂

@smith558 smith558 added the P1 High priority label Feb 14, 2024
@damonholden
Copy link

Also worth mentioning the .with() Array method aswell. Also some of the tasks like "copy and sort" in this section need to be updated to include the new apis.

Would you be interested in opening a PR for this after this one gets merged (will be soon)? 🙂

Sounds good. Once merged, I'll work on the PR to get the other methods in and adjust the tasks as well. Will keep an eye out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 High priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants