Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Apr 3, 2024. It is now read-only.

Conversation

@sofisl
Copy link
Contributor

@sofisl sofisl commented May 17, 2022

Migrate library to Node 12

@sofisl sofisl requested review from a team as code owners May 17, 2022 02:23
@generated-files-bot
Copy link

generated-files-bot bot commented May 17, 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/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
  • .kokoro/presubmit/node8/common.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/presubmit/node8/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 size: m Pull request size is medium. api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. labels May 17, 2022
@codecov
Copy link

codecov bot commented May 17, 2022

Codecov Report

Merging #1084 (502bfc6) into main (fd477ef) will decrease coverage by 0.34%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main    #1084      +/-   ##
==========================================
- Coverage   66.11%   65.77%   -0.35%     
==========================================
  Files          21       21              
  Lines        1750     1750              
  Branches      352      352              
==========================================
- Hits         1157     1151       -6     
- Misses        510      512       +2     
- Partials       83       87       +4     
Impacted Files Coverage Δ
src/agent/oneplatform-controller.ts 75.23% <0.00%> (ø)
src/agent/state/legacy-state.ts 0.00% <0.00%> (ø)
src/agent/v8/legacy-debugapi.ts 0.00% <ø> (ø)
src/agent/debuglet.ts 78.22% <25.00%> (ø)
src/agent/io/scanner.ts 96.34% <100.00%> (ø)
src/agent/v8/inspector-debugapi.ts 84.61% <100.00%> (ø)
src/agent/state/inspector-state.ts 89.35% <0.00%> (-2.78%) ⬇️

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 fd477ef...502bfc6. Read the comment docs.

findResults.errors.forEach(this.logger.warn);
} catch (err) {
this.logger.error('Error scanning the filesystem.', err);
this.logger.error('Error scanning the filesystem.', err as Error);
Copy link
Contributor

Choose a reason for hiding this comment

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

The linter thanks you, as do I.

@mctavish
Copy link
Contributor

Looks like this got stuck waiting for a test status for node 10. Do you know how to reset its expectations? That test result isn't coming in.

@bcoe bcoe merged commit 3434ddd into main May 18, 2022
@bcoe bcoe deleted the migrateToNode12 branch May 18, 2022 20:53
gcf-merge-on-green bot pushed a commit that referenced this pull request Jun 30, 2022
🤖 I have created a release *beep* *boop*
---


## [7.0.0](v6.0.1...v7.0.0) (2022-06-25)


### ⚠ BREAKING CHANGES

* upgrade to stable (#1088)
* update library to use Node 12 (#1084)

### Features

* upgrade to stable ([#1088](#1088)) ([792ef81](792ef81))


### Bug Fixes

* **deps:** update dependency @google-cloud/common to v4 ([#1090](#1090)) ([f4f6ad4](f4f6ad4))
* **deps:** update dependency firebase-admin to v10 ([#1080](#1080)) ([957ea3c](957ea3c))
* **deps:** update dependency gcp-metadata to v5 ([#1068](#1068)) ([20ff5c5](20ff5c5))


### Build System

* update library to use Node 12 ([#1084](#1084)) ([3434ddd](3434ddd))

---
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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.