no-param-reassign with {"props": true} should catch uses of Object.assign

Tell us about your environment

• ESLint Version: 3.9.1

• Node Version: 6.9.1

• npm Version: 3.10.8

What parser (default, Babel-ESLint, etc.) are you using? default

Please show your full configuration:

The only relevant part:

    "no-param-reassign": [2, {"props": true}],

What did you do? Please include the actual source code causing the issue.

function overwriteParamUsingObjectAssign(obj) {
  Object.assign(obj, {foo: 1});
}

What did you expect to happen? An error should be triggered, since properties of obj are being overwritten using Object.assign.

What actually happened? Please include the actual, raw output from ESLint. No error was triggered.

[not-an-aardvark]

My opinion is that since Object.assign is a native language feature, it might warrant special treatment that we wouldn’t give to userland functions such as lodash.assign.

[not-an-aardvark]

Thanks for your interest in improving ESLint. Unfortunately, it looks like this issue didn’t get enough support from the team and so I’m closing it. While we wish we’d be able to accommodate everyone’s requests, we do need to prioritize. We’ve found that issues failing to reach consensus after a long time tend to never do it, and as such, we close those issues. This doesn’t mean the idea isn’t interesting, just that it’s not something the team can commit to.