-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue #14064: Regexp: violation pointing to line 0 while other Checks count first line as 1 #14057
Conversation
cf86aa5
to
f20ddcb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only two places s left, you caught all - good.
suppression should not be removed, as there are places where it is required.
items for both:
...est/resources/com/puppycrawl/tools/checkstyle/checks/regexp/regexp/InputRegexpSemantic2.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Items
...est/resources/com/puppycrawl/tools/checkstyle/checks/regexp/regexp/InputRegexpSemantic2.java
Outdated
Show resolved
Hide resolved
...urces/com/puppycrawl/tools/checkstyle/checks/regexp/regexp/InputRegexpTrailingComment11.java
Outdated
Show resolved
Hide resolved
b1abbd8
to
8d89782
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Items
...urces/com/puppycrawl/tools/checkstyle/checks/regexp/regexp/InputRegexpTrailingComment11.java
Outdated
Show resolved
Hide resolved
special defect in Check is required to be fixed to let us use good style of test, this migration actaully found a defect in Check pitest will fail, we need to extend test to kill such mutations. After issue is approved we need to change link to issue in commit to new issue, as it is actual fix in Check. |
Issue is approved, we need to use new issue in commit as we are fixing defect. https://pitest.org/quickstart/mutators/#NON_VOID_METHOD_CALLS it simply means we do not have test for execution of Check in default mode, good catch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok to merge
Ok to merge, CI is restarted |
Resolves #14064
covers task of Issue #11446