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

Improve file format handling #64

Closed
totocaca123 opened this issue Sep 12, 2022 · 2 comments
Closed

Improve file format handling #64

totocaca123 opened this issue Sep 12, 2022 · 2 comments
Labels
enhancement Enhancement of existing functionality

Comments

@totocaca123
Copy link
Collaborator

What feature do you want to see added?

For the publishReport, I suggest to take inspiration from warning ng plugin.
For now the main format to support is csv, json, yaml (with specific syntax). Maybe some future files will be supported

  • Generic CSV (simple file format accessible)
    publishReport(tools: [simpleCsv(id: 'customId', name: 'customName', pattern: '*.csv')])

  • json + yaml + xml (plugin internal data format)
    publishReport(tools: [nativeJson(id: 'customId', name: 'customName', pattern: '.json')])
    publishReport(tools: [nativeYaml(id: 'customId', name: 'customName', pattern: '
    .yaml')])
    publishReport(tools: [nativeXml(id: 'customId', name: 'customName', pattern: '*.xml')])

In the future (with tool popularity increasing), other format might be supported (file from specific standards for metric data)

Upstream changes

No response

@totocaca123 totocaca123 added the enhancement Enhancement of existing functionality label Sep 12, 2022
@simonsymhoven
Copy link
Member

Thanks! Thats exactly what i am doing right now! I am now ready to take care of the merging of multiple reports..

@simonsymhoven
Copy link
Member

@totocaca123 For now I have the following api:

publishReport name: "JSON Report", displayType: "dual", provider: json(pattern: "etc/*.json")
publishReport name: "XML Report", displayType: "dual", provider: xml(pattern: "etc/*.xml")
publishReport name: "YAML Report", displayType: "dual", provider: yaml(pattern: "etc/*.yaml")

the whole infrastructure is in place, now I have to merge the reports and plot the charts. And then let's see where errors still appear...

Bildschirmfoto 2022-09-13 um 20 28 47

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of existing functionality
Projects
None yet
Development

No branches or pull requests

2 participants