7$: warn on 'x == a || null' and 'x == a || undefined'

[strager]

[strager]

I think we can tweak our existing logic for E0190.

[strager]

For this task, you can look at the logic for either:

• E0286/E0287 (parser::error_on_pointless_string_compare in src/quick-lint-js/fe/parse.cpp), introduced in commit 35207e6616267c6c81be63f47ce97ec2452d60df, or
• E0190 (parser::error_on_sketchy_condition in src/quick-lint-js/fe/parse.cpp), introduced in 1634b9808e0785ca75dbf4fb545a40ee403ebc51.

The logic and testing for this lint should be similar to the above lints.

Tests would go in test/test-parse-warning.cpp.

[rohitsasi720]

I can work on this issue

[strager]

@rohitsasi720 Do you need any help?

[HarshitAghera]

@strager , Can I take up this issue?

[strager]

@HarshitAghera I suggest finishing issue #852 first before working on this issue.

@rohitsasi720 Have you made progress on this task?

[HarshitAghera]

@strager, can I start working on this?

[strager]

@HarshitAghera Yes you may!