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

Added --html option to save HTML report #1637

Merged
merged 4 commits into from
Nov 24, 2020

Conversation

rloomans
Copy link
Contributor

This is a fork of @vsapronov's PR #1611, with the addition of a test.

@codecov
Copy link

codecov bot commented Nov 24, 2020

Codecov Report

Merging #1637 (f9854dd) into master (03b6df4) will decrease coverage by 0.14%.
The diff coverage is 90.38%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1637      +/-   ##
==========================================
- Coverage   80.57%   80.42%   -0.15%     
==========================================
  Files          29       30       +1     
  Lines        2656     2677      +21     
  Branches      410      411       +1     
==========================================
+ Hits         2140     2153      +13     
- Misses        419      427       +8     
  Partials       97       97              
Impacted Files Coverage Δ
locust/main.py 20.25% <20.00%> (-0.01%) ⬇️
locust/html.py 97.72% <97.72%> (ø)
locust/argument_parser.py 77.58% <100.00%> (+0.19%) ⬆️
locust/web.py 90.38% <100.00%> (-1.18%) ⬇️
locust/user/task.py 95.74% <0.00%> (-0.54%) ⬇️
locust/runners.py 84.80% <0.00%> (-0.39%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 03b6df4...f9854dd. Read the comment docs.

@cyberw
Copy link
Collaborator

cyberw commented Nov 24, 2020

thanks! (and thanks @vsapronov)

@cyberw cyberw merged commit 0e362af into locustio:master Nov 24, 2020
@rloomans rloomans deleted the save-html-report branch November 24, 2020 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants