cleverage / eslint-config

eslint config for Clever Age projects
MIT License
2 stars 1 forks source link

no-param-reassign #7

Open ryuran opened 5 years ago

ryuran commented 5 years ago

Currently

https://github.com/airbnb/javascript/blob/070e6200bb6c70fa31470ed7a6294f2497468b44/packages/eslint-config-airbnb-base/rules/best-practices.js#L174-L190

    // disallow reassignment of function parameters
    // disallow parameter object manipulation except for specific exclusions
    // rule: https://eslint.org/docs/rules/no-param-reassign.html
    'no-param-reassign': ['error', {
      props: true,
      ignorePropertyModificationsFor: [
        'acc', // for reduce accumulators
        'accumulator', // for reduce accumulators
        'e', // for e.returnvalue
        'ctx', // for Koa routing
        'req', // for Express requests
        'request', // for Express requests
        'res', // for Express responses
        'response', // for Express responses
        '$scope', // for Angular 1 scopes
      ]
    }],

Proposal

'no-param-reassign': ['error', { props: false }],

https://eslint.org/docs/rules/no-param-reassign