Skip to content

DefaultCheckstyleExecutor should be a failure, not an error #656

@elharo

Description

@elharo

Affected version

HEAD

Bug description

Resolve this todo in DefaultCheckstyleExecutor

            if (request.isFailsOnError()) {
                // TODO: should be a failure, not an error. Report is not meant to
                // throw an exception here (so site would
                // work regardless of config), but should record this information
                throw new CheckstyleExecutorException(message.toString());
            } 

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions