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

Extend ReportMergeTask for md #5034

Open
kvn-stgl opened this issue Jul 4, 2022 · 6 comments
Open

Extend ReportMergeTask for md #5034

kvn-stgl opened this issue Jul 4, 2022 · 6 comments
Labels
feature gradle-plugin never gets stale Mark Issues/PRs that should not be closed as they won't get stale

Comments

@kvn-stgl
Copy link
Contributor

kvn-stgl commented Jul 4, 2022

Expected Behavior

I would like to use the Markdown Report (#4858) in projects with multiple modules.

For this, I would like to extend the ReportMergeTask to work also with markdown files. Sadly, this is kind of a problem: We can not simply parse the markdown reports because it's just text.

Idea

An idea would be to implement a new output generator like SerializedOutput which just serializes the Detektion-object and extend the ReportMergeTask to generate an MD file out of the serialized Detektion-object.

I'm not really happy with this solution but I haven't another idea about how to get a merged markdown report (and maybe we can also use this for the txt/html output).

Your Environment

Version of detekt used: 1.21.0-RC2
Version of Gradle used (if applicable): 7.4.2

@3flex
Copy link
Member

3flex commented Jul 4, 2022

Related to #4827

@BraisGabin
Copy link
Member

Also related with #4289. If the Detekt task generates a single report or serializes the Detektion list this and other tasks would be more easy. Probably SARIF could be that base report.

The idea detekt-core > SARIF? > Reports (single, merged, baselines, etc)

This would work extremely well with gradle. But this is a huge refactor.

@3flex
Copy link
Member

3flex commented Jul 5, 2022

Yeah that's basically what JaCoCo does, as an example.

@github-actions
Copy link

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.

@github-actions github-actions bot added the stale label Oct 30, 2022
@BraisGabin BraisGabin removed the stale label Nov 6, 2022
@github-actions
Copy link

github-actions bot commented Feb 5, 2023

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.

@github-actions github-actions bot added the stale label Feb 5, 2023
@3flex 3flex added never gets stale Mark Issues/PRs that should not be closed as they won't get stale and removed stale labels Feb 5, 2023
@JavierSegoviaCordoba
Copy link
Contributor

Html is not working too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature gradle-plugin never gets stale Mark Issues/PRs that should not be closed as they won't get stale
Projects
None yet
Development

No branches or pull requests

4 participants