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.

Sign up for GitHub

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

Jump to bottom

Junit reporter #568

Open
mrdannael opened this issue Mar 22, 2024 · 2 comments · May be fixed by #586
Open

Junit reporter #568

mrdannael opened this issue Mar 22, 2024 · 2 comments · May be fixed by #586
Labels
feature request Feature request

Comments

@mrdannael
Copy link

Could it be possible to include a JUnit reporter as part of Knip? For example, Bitbucket (and Jenkins, I believe, along with potentially other platforms) is capable of reading JUnit XML reports and displaying errors in a separate pipeline tab for end users. The reporter itself would generate an XML file containing all caught unused dependencies, unused exports, etc.

If that's acceptable to you, I can prepare a pull request for this feature/reporter. 😊
@mrdannael mrdannael added the feature request Feature request label Mar 22, 2024
@mrdannael mrdannael changed the title Junit reporter for CI Junit reporter Mar 22, 2024
@webpro
Copy link
Owner

webpro commented Mar 22, 2024

Yes, that's interesting :)

You can already use custom reporters, but would be great to include them with Knip for others to enjoy as well.

@mrdannael mrdannael linked a pull request Apr 7, 2024 that will close this issue
Open
@voxpelli
Copy link
Sponsor Contributor

voxpelli commented Apr 7, 2024

Similar one used by eg GitHub: https://docs.github.com/en/code-security/code-scanning/integrating-with-code-scanning/sarif-support-for-code-scanning

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request Feature request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add JUnit reporter mrdannael/knip
3 participants
@voxpelli @webpro @mrdannael