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

chore: do not specify @google-cloud/profiler in test package.json#845

Merged
nolanmar511 merged 1 commit intomainfrom
nolanmar511-patch-1
Jul 6, 2022
Merged

chore: do not specify @google-cloud/profiler in test package.json#845
nolanmar511 merged 1 commit intomainfrom
nolanmar511-patch-1

Conversation

@nolanmar511
Copy link
Copy Markdown
Contributor

@nolanmar511 nolanmar511 commented Jul 6, 2022

The benchmark test should use the version of @google-cloud/profiler at head on GitHub; specifying the version within this package.json seems potentially misleading (though may not be causing issue).

The benchmark test should use the version of @google-cloud/profiler at head on GitHub, so should not be specified in this package.json
@nolanmar511 nolanmar511 requested review from a team July 6, 2022 22:14
@product-auto-label product-auto-label Bot added size: xs Pull request size is extra small. api: cloudprofiler Issues related to the googleapis/cloud-profiler-nodejs API. labels Jul 6, 2022
@nolanmar511 nolanmar511 changed the title chore: do not specify @google-cloud/profiler in test package.json DO NOT REVIEW chore: do not specify @google-cloud/profiler in test package.json Jul 6, 2022
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 6, 2022

Codecov Report

Merging #845 (850e859) into main (be24c78) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #845   +/-   ##
=======================================
  Coverage   68.87%   68.87%           
=======================================
  Files           7        7           
  Lines        1253     1253           
  Branches       58       58           
=======================================
  Hits          863      863           
  Misses        389      389           
  Partials        1        1           

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 be24c78...850e859. Read the comment docs.

@nolanmar511 nolanmar511 changed the title DO NOT REVIEW chore: do not specify @google-cloud/profiler in test package.json chore: do not specify @google-cloud/profiler in test package.json Jul 6, 2022
@nolanmar511 nolanmar511 merged commit 3249ea1 into main Jul 6, 2022
@nolanmar511 nolanmar511 deleted the nolanmar511-patch-1 branch July 6, 2022 23:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: cloudprofiler Issues related to the googleapis/cloud-profiler-nodejs API. size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants