allow functions as first argument to Object.assign

[davidgoli]

PR checklist

• Addresses an existing issue: prefer-object-spread should ignore use with functions #3055
• New feature, bugfix, or enhancement
  • Includes tests
• Documentation update

Overview of change:

Permits functions as first argument to Object.assign

Is there anything you'd like reviewers to focus on?

CHANGELOG.md entry:

• [bugfix] prefer-object-spread Permit functions as first argument to Object.assign

[palantirtech]

Thanks for your interest in palantir/tslint, @davidgoli! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

[ajafff]

Please move this line above the comment

[davidgoli]

done

[ajafff]

Thanks @davidgoli

[wesleymostien]

only works for anonymous functions ?

Doesn't complain here :

export default Object.assign((args) => {
  cons.log(args);
}, log);

But does complain here :

function jos (args) {
  cons.log(args);
}
export default Object.assign(jos, log);

Is this a bug or by design ?