Skip to content

Conversation

@JimBobSquarePants
Copy link
Member

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following matches the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

Just some general cleanup of the benchmark code.

@JimBobSquarePants JimBobSquarePants changed the title Js/cleanup benchmarks Benchmark Code Quality Cleanup Dec 17, 2020
@codecov
Copy link

codecov bot commented Dec 17, 2020

Codecov Report

Merging #1479 (7171079) into master (15476d3) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1479   +/-   ##
=======================================
  Coverage   83.48%   83.48%           
=======================================
  Files         741      741           
  Lines       32575    32575           
  Branches     3652     3652           
=======================================
  Hits        27194    27194           
  Misses       4668     4668           
  Partials      713      713           
Flag Coverage Δ
unittests 83.48% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 15476d3...7171079. Read the comment docs.

@JimBobSquarePants JimBobSquarePants merged commit 611ff85 into master Dec 17, 2020
@JimBobSquarePants JimBobSquarePants deleted the js/cleanup-benchmarks branch December 17, 2020 12:40
JimBobSquarePants added a commit that referenced this pull request Mar 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant