jellyfin-web Replace "Object.assign()" With an Object Spread

Replace "Object.assign()" With an Object Spread

Open Adam4376 opened this issue 1 year ago • 5 comments

This is a change requested by this issue on Sonar Cloud

Nov 28 '23 00:11 Adam4376

Typically what I look for in these type of cleanup PRs is that they fix all instances of the issue and add an eslint rule (if possible) to prevent new instances from being added.

Thanks!

Nov 29 '23 18:11 thornbill

Typically what I look for in these type of cleanup PRs is that they fix all instances of the issue and add an eslint rule (if possible) to prevent new instances from being added.
* [all reported violations of this rule](https://sonarcloud.io/project/issues?resolved=false&rules=javascript%3AS6661&id=jellyfin_jellyfin-web)

* [eslint rule](https://eslint.org/docs/latest/rules/prefer-object-spread)
Thanks!

I believe I've changed all the usages. Added the eslint rule as well.

Dec 18 '23 19:12 Adam4376