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

Replace assert.equal with assert.strictEqual#562

Merged
mayurkale22 merged 2 commits into
census-instrumentation:masterfrom
mayurkale22:strict-equal
May 29, 2019
Merged

Replace assert.equal with assert.strictEqual#562
mayurkale22 merged 2 commits into
census-instrumentation:masterfrom
mayurkale22:strict-equal

Conversation

@mayurkale22

Copy link
Copy Markdown
Member

assert.equal() is deprecated since node v10.

There will be follow-up PRs,

  1. Replace assert.notEqual with assert.notstrictEqual
  2. Replace assert.deepEqual with assert.deepStrictEqual
  3. Replace assert.notDeepEqual with assert.notStrictEqual

@codecov-io

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #562      +/-   ##
==========================================
- Coverage    95.3%   95.27%   -0.03%     
==========================================
  Files         148      148              
  Lines       10589    10550      -39     
  Branches      744      742       -2     
==========================================
- Hits        10092    10052      -40     
- Misses        497      498       +1
Impacted Files Coverage Δ
src/zpages-frontend/latency-bucket-boundaries.ts 70.42% <0%> (-2.82%) ⬇️
src/stackdriver-monitoring.ts 77.02% <0%> (-2.71%) ⬇️
test/test-root-span.ts 100% <0%> (ø) ⬆️
src/resource-labels.ts 100% <0%> (ø) ⬆️
test/test-metric-component.ts 100% <0%> (ø) ⬆️
test/test-stackdriver-cloudtrace-utils.ts 100% <0%> (ø) ⬆️
test/test-tracer-base.ts 100% <0%> (ø) ⬆️
test/test-prometheus-stats.ts 100% <0%> (ø) ⬆️
test/test-zpages.ts 99.27% <0%> (ø) ⬆️
test/test-metric-registry.ts 100% <0%> (ø) ⬆️
... and 2 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 a157106...342a71b. Read the comment docs.

@mayurkale22 mayurkale22 merged commit 20e4cf0 into census-instrumentation:master May 29, 2019
@mayurkale22 mayurkale22 deleted the strict-equal branch May 29, 2019 04:44
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