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

Support official HTML reporter #312

Closed
rudym87 opened this issue Dec 6, 2019 · 5 comments · Fixed by #318
Closed

Support official HTML reporter #312

rudym87 opened this issue Dec 6, 2019 · 5 comments · Fixed by #318

Comments

@rudym87
Copy link

rudym87 commented Dec 6, 2019

Since the HTML reporter is now integrated in ktlint 0.36.0, it should be added to the configurable ones.

Right now

ktlint {
    reporters {
        reporter "html"
    }
}

produces

> Cannot convert string value 'html' to an enum value of type 'org.jlleitschuh.gradle.ktlint.reporter.ReporterType' (valid case insensitive values: PLAIN, PLAIN_GROUP_BY_FILE, CHECKSTYLE, JSON)
@smuga
Copy link

smuga commented Dec 6, 2019

Before it'll be available, you still can use HTML reporter by adding this:

customReporters {
    "html" {
        fileExtension = "html"
        dependency = "com.pinterest.ktlint:ktlint-reporter-html:0.36.0"
    }
}

@saket
Copy link

saket commented Jan 1, 2020

@smuga I get this error when trying to add that custom reporter:

* What went wrong:
A problem occurred evaluating root project 'Press'.
> Supplied String module notation 'com.pinterest.ktlint:ktlint-reporter-html:0:36:0' is invalid. Example notations: 'org.gradle:gradle-core:2.2', 'org.mockito:mockito-core:1.9.5:javadoc'.

@smuga
Copy link

smuga commented Jan 2, 2020

@saket Sorry, version number should be 0.36.0, not 0:36:0 ;) I've updated previous post.

@ImMathanR
Copy link

Any plan on when the HTML support would be released?

@Tapchicoma
Copy link
Collaborator

@ImMathanR We are planning to do new release this week, worth case beginning of the next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants