Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build!: update library to use Node 12 #835

Merged
merged 43 commits into from
Jun 1, 2022
Merged

build!: update library to use Node 12 #835

merged 43 commits into from
Jun 1, 2022

Conversation

sofisl
Copy link
Contributor

@sofisl sofisl commented May 16, 2022

Migrate library to Node 12

@sofisl sofisl requested review from a team as code owners May 16, 2022 20:14
@product-auto-label product-auto-label bot added the size: m Pull request size is medium. label May 16, 2022
@generated-files-bot
Copy link

generated-files-bot bot commented May 16, 2022

Warning: This pull request is touching the following templated files:

  • .kokoro/continuous/node10/common.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/continuous/node10/docs.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/continuous/node10/test.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/continuous/node8/common.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/continuous/node8/docs.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/continuous/node8/lint.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/continuous/node8/samples-test.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/continuous/node8/system-test-grpcjs.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/continuous/node8/system-test.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/continuous/node8/test.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/presubmit/node10/common.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/presubmit/node10/docs.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/presubmit/node10/lint.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/presubmit/node10/test.cfg - .kokoro files are templated and should be updated in synthtool
  • .github/workflows/ci.yaml - .github/workflows/ci.yaml (GitHub Actions) should be updated in synthtool

@product-auto-label product-auto-label bot added the api: cloudprofiler Issues related to the googleapis/cloud-profiler-nodejs API. label May 16, 2022
@codecov
Copy link

codecov bot commented May 17, 2022

Codecov Report

Merging #835 (9da5d65) into main (74e287d) will decrease coverage by 0.27%.
The diff coverage is 11.11%.

@@            Coverage Diff             @@
##             main     #835      +/-   ##
==========================================
- Coverage   69.15%   68.87%   -0.28%     
==========================================
  Files           7        7              
  Lines        1248     1253       +5     
  Branches       58       58              
==========================================
  Hits          863      863              
- Misses        384      389       +5     
  Partials        1        1              
Impacted Files Coverage Δ
src/profiler.ts 72.18% <11.11%> (-0.63%) ⬇️

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 74e287d...9da5d65. Read the comment docs.

@sofisl sofisl added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 18, 2022
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 18, 2022
@product-auto-label product-auto-label bot removed the size: m Pull request size is medium. label May 20, 2022
@sofisl sofisl added the automerge Merge the pull request once unit tests and other checks pass. label Jun 1, 2022
@sofisl sofisl removed the automerge Merge the pull request once unit tests and other checks pass. label Jun 1, 2022
@sofisl sofisl merged commit 07221ef into main Jun 1, 2022
@sofisl sofisl deleted the migrateToNode12 branch June 1, 2022 19:11
gcf-merge-on-green bot pushed a commit that referenced this pull request Jun 30, 2022
🤖 I have created a release *beep* *boop*
---


## [5.0.0](v4.2.0...v5.0.0) (2022-06-20)


### ⚠ BREAKING CHANGES

* update library to use Node 12 (#835)

### Bug Fixes

* **deps:** update dependency @google-cloud/common to v4 ([#843](#843)) ([1af6b0f](1af6b0f))
* **deps:** update dependency @google-cloud/logging-min to v10 ([#838](#838)) ([1120298](1120298))


### Build System

* update library to use Node 12 ([#835](#835)) ([07221ef](07221ef))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
@@ -20,5 +23,7 @@
"posttest": "npm run check"
},
"devDependencies": {},
"dependencies": {}
"dependencies": {
"@google-cloud/profiler": "^4.2.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this change means the system test will always run with @google-cloud/profiler 4.2.0?

If so, that would mean the system test won't work anymore

grayside pushed a commit to GoogleCloudPlatform/nodejs-docs-samples that referenced this pull request Nov 11, 2022
🤖 I have created a release *beep* *boop*
---


## [5.0.0](googleapis/cloud-profiler-nodejs@v4.2.0...v5.0.0) (2022-06-20)


### ⚠ BREAKING CHANGES

* update library to use Node 12 (#835)

### Bug Fixes

* **deps:** update dependency @google-cloud/common to v4 ([#843](googleapis/cloud-profiler-nodejs#843)) ([1af6b0f](googleapis/cloud-profiler-nodejs@1af6b0f))
* **deps:** update dependency @google-cloud/logging-min to v10 ([#838](googleapis/cloud-profiler-nodejs#838)) ([1120298](googleapis/cloud-profiler-nodejs@1120298))


### Build System

* update library to use Node 12 ([#835](googleapis/cloud-profiler-nodejs#835)) ([07221ef](googleapis/cloud-profiler-nodejs@07221ef))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
ace-n pushed a commit to GoogleCloudPlatform/nodejs-docs-samples that referenced this pull request Nov 12, 2022
🤖 I have created a release *beep* *boop*
---


## [5.0.0](googleapis/cloud-profiler-nodejs@v4.2.0...v5.0.0) (2022-06-20)


### ⚠ BREAKING CHANGES

* update library to use Node 12 (#835)

### Bug Fixes

* **deps:** update dependency @google-cloud/common to v4 ([#843](googleapis/cloud-profiler-nodejs#843)) ([1af6b0f](googleapis/cloud-profiler-nodejs@1af6b0f))
* **deps:** update dependency @google-cloud/logging-min to v10 ([#838](googleapis/cloud-profiler-nodejs#838)) ([1120298](googleapis/cloud-profiler-nodejs@1120298))


### Build System

* update library to use Node 12 ([#835](googleapis/cloud-profiler-nodejs#835)) ([07221ef](googleapis/cloud-profiler-nodejs@07221ef))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: cloudprofiler Issues related to the googleapis/cloud-profiler-nodejs API. size: l Pull request size is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants