Add new linter rule: avoid comparison assersion

[nunnatsa]

Description

A worng comparison is a boolean comparison using ==',!=,>,>=,<or<=`, and assertion of a boolean value.

Examples:

Expect(x == 10).Should(BeTrue()) ==> Expect(x).Should(Equal(10))

Expect(s != "testing").Should(Equal(true)) ==> Expect(s).ShouldNot(Equal("testing"))

Expect(x > 10).To(BeTrue()) ==> Expect(x).To(BeNumerically(">", 10))

Added a new executable's flag to suppress this new rule: --suppress-compare-assertion=true

Fixes #64

Type of change

Please delete options that are not relevant.

• [x] New feature (non-breaking change which adds functionality)
• [x] This change requires a documentation update

How Has This Been Tested?

• [x] Added test case and related test data
• [x] Update the functional test

Checklist:

• [x] I have performed a self-review of my own code
• [x] I have made corresponding changes to the documentation
• [x] My changes generate no new warnings
• [x] I have added tests that prove my fix is effective or that my feature works
• [x] New and existing unit tests pass locally with my changes
• [x] Any dependent changes have been merged and published in downstream modules
• [x] I ran golangci-lint

@pohly

[coveralls]

Pull Request Test Coverage Report for Build 4279981495

• 132 of 155 (85.16%) changed or added relevant lines in 3 files are covered.
• 2 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-1.6%) to 86.63%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
types/config.go	6	7	85.71%
ginkgo_linter.go	121	143	84.62%
<!--	Total:	132	155	85.16%	-->

Files with Coverage Reduction	New Missed Lines	%
ginkgo_linter.go	1	88.06%
types/config.go	1	32.43%
<!--	Total:	2		-->

Totals
Change from base Build 4223637700:	-1.6%
Covered Lines:	635
Relevant Lines:	733

---

💛 - Coveralls

[nunnatsa]

@pohly - would you mind to take a look on this PR?