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

Reduce data volume in reports to reduce file size and make them less verbose #963

Open
3 of 9 tasks
BioPhoton opened this issue Mar 5, 2025 · 2 comments
Open
3 of 9 tasks

Comments

@BioPhoton
Copy link
Collaborator

BioPhoton commented Mar 5, 2025

User story

As a user of the CLI or CI actions I want to have small and concise reports by default. At the moment the reports (terminal, MD, JSON) are verbose and big in file size.

Acceptance criteria

  • The terminal report of utils package needs to accept options to print a reduced report by default
  • The MD as well as JSON reports of the utils package needs to accept options to create a reduced report by default
  • The GitHub action needs to accept options to create a reduced diff report by default

Implementation details

Delivery plan:

@vmasek
Copy link
Collaborator

vmasek commented Mar 6, 2025

On refinement we had a conclusion on following proposal:

Let's open up the persist in a way that we'll allow hooking in you own reporters, in similar fashion other tools do, especially the ones that report a lot like testing or coverage (jest, vitest, cypress, ...).

I image it in a way that persist-config will be type-safe for the baked in reporters that are json and md at the moment.

  • We will not reduce them as they fulfill their use-case pretty well.
  • We can however decide to ship reduced ones, for example your proposed md-only-failing or other that make sense to keep in our codebase as they are not too specific.
  • We will create an API that will allow you to hook your own reported that can do anything it likes as it will have the same input for post-processing as baked in ones.

Reporter naming note: I'd suggest we enforce custom-* or some kind of prefix for custom reporter names, just so we don't end up in collision with baked in one.

@vmasek
Copy link
Collaborator

vmasek commented Mar 6, 2025

Closed the reduced JSON reporters proposals as they will be supported by having custom reporter to post process to whatever format or reduced data structure user desires.

Keeping the reduced markdown reporters as they can be interesting to maintain in this repo, just as options for users to see less cluttered reports.

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

No branches or pull requests

2 participants