Skip to content
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

RegexpSinglelineCheck is broken #1079

Open
olegmoz opened this issue Sep 18, 2020 · 2 comments
Open

RegexpSinglelineCheck is broken #1079

olegmoz opened this issue Sep 18, 2020 · 2 comments
Labels
Milestone

Comments

@olegmoz
Copy link

olegmoz commented Sep 18, 2020

I've got a Java class with string literal "/**/". For some reason RegexpSinglelineCheck rule violation is reported for it with the following message:

This kind of comment is not allowed. (RegexpSinglelineCheck)

It also impossible to disable the rule using @checkstyle RegexpSinglelineCheck (X lines) in JavaDoc.
"/**/" string literal does not has anything to do with RegEx and is not a comment, so it should not cause any errors.

Version of Qulice Maven Plugin is 0.18.19

@elbraulio
Copy link

elbraulio commented Oct 12, 2022

I have this

/**
     * Ctor.
     *
     * @param line    Line position.
     * @param column  Column position.
     * @param message Error message.
     */
    public ParseError(
        final Integer line,
        final Integer column,
        final String message
    ) {

and got @param tag description should start with capital letter (RegexpSinglelineCheck)
for the first two params, the third one is ok.

as I see that something is wrong, so I tried to use

// @checkstyle RegexpSinglelineJava (50 lines)

/**
   * Ctor.
   *
   * @param line    Line position.
   * @param column  Column position.
   * @param message Error message.
   */

but the problem still pops up when running the build

I'm using version 0.22.0 on Java 17

@elbraulio
Copy link

I downgraded to 0.19.5 and worked fine

@yegor256 yegor256 added the bug label Nov 24, 2022
@yegor256 yegor256 added this to the next milestone Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants