`no-self-assign` failing for property assignment

What version of ESLint are you using? 3.1.1
What parser (default, Babel-ESLint, etc.) are you using? Default
Please show your full configuration: { "env": { "browser": true }, "parserOptions": { "ecmaVersion": 6 }, "rules": { "no-self-assign": "error" } }
What did you do? Please include the actual source code causing the issue.

var a = 123;
a = a;

var b = { a };
b.a = b.a; 

What did you expect to happen? I would expect b.a = b.a to be a error;
What actually happened? Please include the actual, raw output from ESLint. Only a.a is an error.

2:5 error ‘a’ is assigned to itself no-self-assign

✖ 1 problem (1 error, 0 warnings)

[platinumazure]

Can we put it behind a rule option so users can make that call?

On Jul 21, 2016 11:12 AM, “Nicholas C. Zakas” [e-mail] wrote:

I think it’s good for default behavior since it does not expect that self-assignments have side-effects, normally.

My question is, can we assume that? I’m not sure we can.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/eslint/eslint/issues/6718#issuecomment-234303607, or mute the thread https://github.com/notifications/unsubscribe-auth/AARWemfF03LpMmHU-ynqSqglzbW_cmdtks5qX5phgaJpZM4JQXbi .

[ilyavolodin]

👍