Removing additional repro test step

microsoft / onefuzz

A self-hosted Fuzzing-As-A-Service platform

MIT License

2.82k stars 199 forks source link

Removing additional repro test step #3568

Closed nharper285 closed 9 months ago

nharper285 commented 9 months ago

Summary of the Pull Request

What is this about? Didn't clear all of repro steps.

PR Checklist

[ ] Applies to work item: #xxx
[ ] CLA signed. If not, go over here and sign the CLI.
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

codecov-commenter commented 9 months ago

Codecov Report

Merging #3568 (569dd5e) into main (f251282) will decrease coverage by 7.79%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3568      +/-   ##
==========================================
- Coverage   38.82%   31.03%   -7.79%     
==========================================
  Files         302      136     -166     
  Lines       36950    14747   -22203     
==========================================
- Hits        14344     4577    -9767     
+ Misses      22606    10170   -12436

see 166 files with indirect coverage changes