Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

build!: update library to use Node 12 #1084

Merged
merged 7 commits into from
May 18, 2022
Merged

build!: update library to use Node 12 #1084

merged 7 commits into from
May 18, 2022

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.

@@ -388,7 +388,7 @@ export class Debuglet extends EventEmitter {
findResults = await Debuglet.findFiles(this.config, gaeId);
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.

None yet

3 participants