{"action":"created","issue":{"url":"https://api.github.com/repos/airbnb/javascript/issues/626","labels_url":"https://api.github.com/repos/airbnb/javascript/issues/626/labels{/name}","comments_url":"https://api.github.com/repos/airbnb/javascript/issues/626/comments","events_url":"https://api.github.com/repos/airbnb/javascript/issues/626/events","html_url":"https://github.com/airbnb/javascript/issues/626","id":122620120,"number":626,"title":"no-param-reassign with props","user":{"login":"growlsworth","id":135461,"avatar_url":"https://avatars.githubusercontent.com/u/135461?v=3","gravatar_id":"","url":"https://api.github.com/users/growlsworth","html_url":"https://github.com/growlsworth","followers_url":"https://api.github.com/users/growlsworth/followers","following_url":"https://api.github.com/users/growlsworth/following{/other_user}","gists_url":"https://api.github.com/users/growlsworth/gists{/gist_id}","starred_url":"https://api.github.com/users/growlsworth/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/growlsworth/subscriptions","organizations_url":"https://api.github.com/users/growlsworth/orgs","repos_url":"https://api.github.com/users/growlsworth/repos","events_url":"https://api.github.com/users/growlsworth/events{/privacy}","received_events_url":"https://api.github.com/users/growlsworth/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-16T22:40:40Z","updated_at":"2015-12-16T22:42:02Z","closed_at":null,"body":"Hey AirBnB,\r\nWhat are your thoughts about preventing object manipulation on params as well:\r\nhttp://eslint.org/docs/rules/no-param-reassign.html. I know I'm surprised when I pass an object to a function and the function manipulates the object.\r\n\r\nThe [current value](https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb/rules/best-practices.js#L77) is set to `2`. What do you think about changing it to `[2, { \"props\": true }]`?\r\n\r\nThanks!"},"comment":{"url":"https://api.github.com/repos/airbnb/javascript/issues/comments/165276651","html_url":"https://github.com/airbnb/javascript/issues/626#issuecomment-165276651","issue_url":"https://api.github.com/repos/airbnb/javascript/issues/626","id":165276651,"user":{"login":"ljharb","id":45469,"avatar_url":"https://avatars.githubusercontent.com/u/45469?v=3","gravatar_id":"","url":"https://api.github.com/users/ljharb","html_url":"https://github.com/ljharb","followers_url":"https://api.github.com/users/ljharb/followers","following_url":"https://api.github.com/users/ljharb/following{/other_user}","gists_url":"https://api.github.com/users/ljharb/gists{/gist_id}","starred_url":"https://api.github.com/users/ljharb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ljharb/subscriptions","organizations_url":"https://api.github.com/users/ljharb/orgs","repos_url":"https://api.github.com/users/ljharb/repos","events_url":"https://api.github.com/users/ljharb/events{/privacy}","received_events_url":"https://api.github.com/users/ljharb/received_events","type":"User","site_admin":false},"created_at":"2015-12-16T22:42:02Z","updated_at":"2015-12-16T22:42:02Z","body":"I think absolutely this should be turned on - I wasn't aware that was even an option.\r\n\r\nA PR that updates the README, and updates the lint rule (including linking to the rule docs), would be very welcome."}}