Define no-null-keyword rule

[myitcv]

Addresses #677

Comments/thoughts welcomed. I've duly referenced the TypeScript repo version on which this is shamelessly based.

[myitcv]

This is not a negative diff; I've simply sorted the contents.

[jkillian]

Can you add this to sample.tslint.json and to the README?

[jkillian]

Actually, before you do that, one area of discussion. We're not sure how to handle rules that conflict with each other (we accidentally ran into this problem about a week ago #711).

This rule would potentially conflict with: #625

[myitcv]

@jkillian added to sample.tslint.json and README updated (rebased at the same time)

Yes, this might potentially conflict with #625 (and indeed others) but I think it's obvious why it would conflict. The problem arises where rules conflict and its not immediately obvious why.

[jkillian]

Given more thought, I'm happy to merge this. I think the solution is to just check for conflicting rules as mentioned in #711. @adidahiya, good with you?

[myitcv]

@jkillian just rebased this (saw your comment in #747)

[myitcv]

@adidahiya @jkillian - just rebased this

[adidahiya]

👍