-
-
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 #13345: Enable example tests for SingleLineJavadocCheckExamples… #14871
Issue #13345: Enable example tests for SingleLineJavadocCheckExamples… #14871
Conversation
@prathm3, , please execute |
14d33a8
to
13ed612
Compare
We need this changes in commit https://dev.azure.com/romanivanovjr/romanivanovjr/_build/results?buildId=21400&view=logs&j=c902ebb4-c9f8-5f09-4e17-ff78fbbc842e&t=9ca98c81-ff64-58f0-9d03-a23ac1c4a111&l=1800 Just put in commit all that are changed on your local after mvn verify. |
b7336e8
to
cf17273
Compare
cf17273
to
2934b40
Compare
@romani , thanks for the hint. Failed some checks with |
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:
package com.puppycrawl.tools.checkstyle.checks.javadoc.singlelinejavadoc; | ||
// xdoc section -- start | ||
public class Example1 { | ||
/** @see Math */ // violation |
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.
we have few possible messages from Check https://checkstyle.org/checks/javadoc/singlelinejavadoc.html#Violation_Messages
it is not clear which violation is here, please add message to this line // violation 'some message'
message is regexp so you can provide only most meanigful part of original message
Moved to #14992 |
Issue: #13345
Enabled tests for SingleLineJavadocCheckExamplesTests.java