Add prefer-node-remove rule

[medusalix]

Resolves #213

[sindresorhus]

Add the rule to the recommended config, the recommended config in the readme, and the rule listing in the readme (The description should be the same as the heading of the documentation file). - https://github.com/sindresorhus/eslint-plugin-unicorn/blob/master/docs/new-rule.md#steps

[medusalix]

Is there still something you'd want me to change? @sindresorhus

[sindresorhus]

I just discovered the other day in a different project that there's something called .parentElement (not node). We should support that too (also needs tests and docs).

[futpib]

Please fix the tests I've added

[medusalix]

@futpib Checking for computed to be false has the downside of allowing this to pass:

foo.parentNode['removeChild'](foo);

Do you know any better alternative to differentiate between parentNode.removeChild(foo) and parentNode[removeChild](foo)?

[futpib]

@medusalix ignoring foo.parentNode['removeChild'](foo); is okay.

[sindresorhus]

Nice work, @medusalix 👍