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

Simplify JSON/HTML report generation #241

Closed
fabricereix opened this issue Jul 21, 2021 · 0 comments · Fixed by #243
Closed

Simplify JSON/HTML report generation #241

fabricereix opened this issue Jul 21, 2021 · 0 comments · Fixed by #243
Labels
enhancement New feature or request
Milestone

Comments

@fabricereix
Copy link
Collaborator

fabricereix commented Jul 21, 2021

Currently, an html report can be generated easily with --html DIR.

But in order to accumulate results for different hurl executions, a json file needs to be used with the option --json.
This makes the options --json and --html not symmetrical, and thus, this is not really intuitive.

In order to accumulate results into an html report, the html report itself should be appended directly, without the need to use the option --json.

The option --append can also be removed. Results can be naturally appended to the JSON/HTML report file if this one already exists.

@fabricereix fabricereix added the enhancement New feature or request label Jul 21, 2021
@fabricereix fabricereix changed the title Make --html options fully independent of --json option Simplify json/html report generation Jul 24, 2021
@fabricereix fabricereix changed the title Simplify json/html report generation Simplify JSON/HTML report generation Jul 24, 2021
@fabricereix fabricereix linked a pull request Jul 24, 2021 that will close this issue
@lepapareil lepapareil added this to the 1.3.0 milestone Oct 12, 2021
@lepapareil lepapareil removed the 1.3.0 label Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants