Add no-identical-title rule (fixes #33)
#97
Conversation
|
|
||
| ## Rule Details | ||
|
|
||
| This rule looks at the rule of every test and test suites. It will report when two test suites or two test cases at the same level of a test suite have the same name. |
There was a problem hiding this comment.
I guess rule should be title or name, right?
There was a problem hiding this comment.
Absolutely. And name should probably be title too.
|
Fixed. By the way, I'd recommend squashing PRs in order to keep the commit history clean. |
|
Merged. Thank you. I also squashed the commits of this PR but I guess my changelog tool pr-log won’t work with this as it looks specifically for merge commits on |
|
You're welcome :) Ah, right. Yeah, should be fine in this case, but that's too bad. Maybe try looking at a different tool to does not rely on merge commits, because a flat commit history is really nice IMO. |
Add
no-identical-titlerule (fixes #33)I decided to name it
no-identical-titleto be "consistent" witheslint-plugin-ava's similar rule. Let me know if you prefer it to beno-duplicate-titleor something else.