Skip to content
This repository was archived by the owner on Oct 3, 2023. It is now read-only.

Replace assert.deepEqual with assert.deepStrictEqual#563

Merged
mayurkale22 merged 1 commit into
census-instrumentation:masterfrom
mayurkale22:deep-strict-equal
May 31, 2019
Merged

Replace assert.deepEqual with assert.deepStrictEqual#563
mayurkale22 merged 1 commit into
census-instrumentation:masterfrom
mayurkale22:deep-strict-equal

Conversation

@mayurkale22

Copy link
Copy Markdown
Member

Continuation of #562

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #563 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #563      +/-   ##
==========================================
- Coverage    95.3%   95.26%   -0.04%     
==========================================
  Files         148      147       -1     
  Lines       10590    10548      -42     
  Branches      745      746       +1     
==========================================
- Hits        10093    10049      -44     
- Misses        497      499       +2
Impacted Files Coverage Δ
test/test-grpc.ts 99.47% <0%> (ø) ⬆️
src/grpc-stats/server-stats.ts 100% <0%> (ø) ⬆️
test/test-stackdriver-cloudtrace-utils.ts 100% <0%> (ø) ⬆️
src/grpc-stats/client-stats.ts 100% <0%> (ø) ⬆️
src/grpc-stats/stats-common.ts 100% <0%> (ø) ⬆️
test/test-stats.ts 100% <0%> (ø) ⬆️
src/grpc-stats/common-distributions.ts

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 20e4cf0...056030f. Read the comment docs.

@mayurkale22

Copy link
Copy Markdown
Member Author

@draffensperger Please review once you have time.

@draffensperger draffensperger left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I must have looked at this and thought I reviewed but didn't finish it!

@mayurkale22

Copy link
Copy Markdown
Member Author

I think I must have looked at this and thought I reviewed but didn't finish it!

Np, thanks for the reviews. This PR is important in order to adopt gts v1 (#506) and fix long pending vulnerabilities #506 (comment).

@mayurkale22 mayurkale22 merged commit dbc173b into census-instrumentation:master May 31, 2019
@mayurkale22 mayurkale22 deleted the deep-strict-equal branch May 31, 2019 17:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants