`Array.copyWithin()` signature is incorrect - Part II

[tchetwin]

Bug Report

In #53334 TypeScript's lib was updated to allow Array#copyWithins 2nd argument start to be optional, to match the MDN documentation at that time.

The MDN documentation has since been updated to match the spec, meaning TypeScript 5.1's types are incorrect.

To remediate: a reversion of #53340 is needed. This might be a slightly breaking change for anyone that adopted TypeScript 5.1 and started (incorrectly) relying on the parameter being optional.

🔎 Search Terms

copyWithin

🕗 Version & Regression Information

• This changed between versions 5.0 and 5.1

[DanielRosenwasser]

Should be a quick and easy PR if you (or others) want to fix it up.

[mattburnand]

I am going to pick this up

[shubhamshd]

@DanielRosenwasser @tchetwin @Mburnand-tech I would also like to give it a try, if it's not against the community values!!!

[mattburnand]

@shubhamshd Go for it! I've struggled to correct the tests so they pass.

[shubhamshd]

@DanielRosenwasser please review the changes in pr and let me know, if anything needs to be corrected, thanks!!!

[shubhamshd]

Apologies team, but is there any scheduled time for the pr to get merged?