Skip to content
This repository was archived by the owner on Feb 20, 2026. It is now read-only.

chore(deps): pin pprof #539

Merged
kalyanac merged 1 commit intogoogleapis:masterfrom
nolanmar511:pin-pprof-nodejs
Sep 11, 2019
Merged

chore(deps): pin pprof #539
kalyanac merged 1 commit intogoogleapis:masterfrom
nolanmar511:pin-pprof-nodejs

Conversation

@nolanmar511
Copy link
Copy Markdown
Contributor

@nolanmar511 nolanmar511 commented Sep 10, 2019

@google-cloud/profiler depends heavily on the pprof module.

We want to be sure that changes to pprof that introduce bugs or problems in @google-cloud/profiler can be caught by @google-cloud/profiler's presubmit tests, rather than immediately impacting @google-cloud/profiler as its running in prod.

This will allow us to concentrate testing in this repository, rather than duplicate all testing here and in https://github.com/google/pprof-nodejs.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 10, 2019
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 10, 2019

Codecov Report

Merging #539 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #539   +/-   ##
=======================================
  Coverage   75.73%   75.73%           
=======================================
  Files           7        7           
  Lines         272      272           
  Branches       56       56           
=======================================
  Hits          206      206           
  Misses         50       50           
  Partials       16       16

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 dfce0ee...1325a20. Read the comment docs.

@kalyanac kalyanac merged commit 470bf3c into googleapis:master Sep 11, 2019
Comment thread package.json
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants