Disallow assignment to exports (node/no-exports-assign) #1400
Description
https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-exports-assign.md
Assigning to the exports variable does not work as expected.
// This assigned object is not exported.
// You need to use `module.exports = { ... }`.
exports = {
foo: 1
}This rule is aimed at disallowing exports = {}, but allows module.exports = exports = {} to avoid conflict with node/exports-style rule's allowBatchAssign option.
👍 Examples of correct code for this rule:
/*eslint node/no-exports-assign: error */
module.exports.foo = 1
exports.bar = 2
module.exports = {}
// allows `exports = {}` if along with `module.exports =`
module.exports = exports = {}
exports = module.exports = {}👎 Examples of incorrect code for this rule:
/*eslint node/no-exports-assign: error */
exports = {}I don't expect this to be controversial