Description
I don't know who thought it's add good idea to check the index range. (Checked history, but only one issue questioned it, and closed by owner)
if (actualIndex < 0 || actualIndex >= len) {
throw new RangeError();
}I agree it's safer, but this never happen in other array methods. (except the size of array is too big)
I haven't try to use it, it just an idea.
Fail
const replacedLastElement = array.with(-1, 'value'); // throws if it's emptyconst clonedAndAppendValue = array.with(array.length, 'value');
// Always throws, have to use old way, though it should be preferred IMO
const clonedAndAppendValue = [...array, 'value'];Pass
Only if .length checked.
if (array.length > known_length) {
newArray = array.with(1, 'value')
}
sindresorhus / eslint-plugin-unicorn
More than 100 powerful ESLint rules
4.2k
How does funding with Polar work?
1
Pay now to fund the work behind this issue.
2
Get updates on progress being made.
3
Maintainer is rewarded once the issue is completed.
FAQ
Backer
You're funding impactful open source efforts
Contributor
You want to contribute to this effort
Maintainer
You want to get funding like this too