migrate code from googleapis/nodejs-error-reporting #2821

kweinmeister · 2022-11-10T13:16:50Z

Error reporting (Error reporting #408)
Update dependencies + fix a few tests (Update dependencies + fix a few tests #448)
Build updates. (Build updates. #462)
Update dependencies (Update dependencies #468)
Complete code migration (Regenerated the kraken app to be a simpler sample. Added links to dep… #18)
chore: fix the samples tests (Added webpack sample. Closes #38. #39)
Fixing vulnerability reported by snyk (Update README.md #51)
chore: update README to reflect the new API (Add datastore gql snippets #76)
chore: release v0.4.0 (Add Cloud Vision face detection sample. #109)
chore: lock files maintenance (Add Missing Dependency #114)
chore: lock files maintenance (Re-organized readme files. #118)
chore: update package locks (Strings! #122)
chore: upgrade to gts 0.6.0 (Incorrect memcache url in app.yaml #123)
chore: lock files maintenance (Add HTTP Cloud Function. #131)
fix: upgrade to the latest @google-cloud/common (Update GCF code samples for new signature of functions with HTTP trigger #128)
fix: update gts and typescript (Switch back to mocha from ava. #143)
chore: update sample lockfiles (Shows how to exclude property from index (for docs) #147)
refactor: drop repo-tool as an exec wrapper (Use gcloud app instead of gcloud preview app #148)
bump version to 0.5.0 (Uses 'properties' region for the excluded property snippet #149)
chore(deps): update dependency sinon to v5.1.1 (Add Cloud Functions Slack (Slash Command) sample. #151)
fix(deps): update dependency yargs to v12 (split unit tests and system tests into separate tasks #154)
chore(deps): update dependency sinon to v6 (Switch CloudSQL sample to 2nd gen. Added unit tests. #153)
chore(deps): update dependency nyc to v12 (can't deploy on app engine #152)
chore(deps): lock file maintenance (Adding missing error parameter to callback #156)
chore(deps): lock file maintenance (Upgraded storage sample #160)
fix: drop support for nodejs 9.x (Add list_datasets_and_projects sample #161)
chore(deps): lock file maintenance (App Engine and Cloud SQL - connection lost issue #163)
chore(deps): lock file maintenance (Pubsub tests #164)
chore(deps): lock file maintenance (Switch to individual API packages. #169)
chore(deps): lock file maintenance (Storage ACL samples. #172)
chore: require node 8 for samples (Prediction API samples #178)
chore(deps): lock file maintenance (Auto-generate command instructions for README.md #181)
chore(deps): lock file maintenance (Update all dependencies 🌴 #191)
chore: ignore package-lock.json (Use stub instead of anonymous function. #193)
chore(deps): update dependency nyc to v13 (Fix typo #200)
chore(deps): update dependency sinon to v7 (bigquery/tables code create table is broken #223)
chore(deps): update dependency @google-cloud/nodejs-repo-tools to v3 (@google-cloud/datastore 0.5.0 does not return entity keys with query results #250)
Update region tags in samples (Update Pub/Sub samples. #259)
refactor(samples): convert sample tests from ava to mocha (Update Speech samples. #257)
Release v0.5.2 (Not working sample: /appengine/websockets #292)
Release v0.6.0 (Add region tags for writing timeseries #298)
fix(deps): update dependency yargs to v13 (Update Vision samples to be easier to read. Update dependencies. #312)
chore(deps): update dependency mocha to v6
Release v0.6.1 (Fix issue with read-only transaction #325)
Release v0.6.2 (Replace node-mailer module with something else #338)
Release v0.6.3 (Update translate dependency #340)
docs(samples): modernize samples, sample tests (Bring ML APIs up to standard. #346)
chore: release 1.0.0 (Split analyze.js into two separate samples #360)
chore: release 1.1.0 (Update App Engine Websockets sample. #378)
chore: release 1.1.1 (Add new video samples #382)
chore: update license headers
chore: release 1.1.3 (Update DLP README #414)
refactor: use explicit mocha imports (speechClient.recognize() giving empty array as response? #425)
chore(deps): update dependency mocha to v7 (Add (missing) sample parameter values #428)
chore(deps): update dependency gaxios to v3 (Remove (likely) unnecessary Express dependencies #457)
build!: update to latest version of gts and typescript (Snippet consistency #467)
chore: release 2.0.0 (Updating Mailjet code samples to match Send API v3.1 config #469)
chore(deps): update dependency mocha to v8 (Add Metadata server sample. #485)
chore(deps): update dependency gaxios to v4 (Update README.md to include region #524)
chore: release 2.0.1 (functions/datastore sample #536)
chore: release 2.0.2 (Fixes script name for HTTP example #573)
chore: release 2.0.3 (Typo in AppEngine/Resify Readme #594)
chore: release 2.0.4 (Fix typo in gcloud command #605)
chore(main): release 2.0.5 (Add missing lint dependency #636)
chore(deps): update dependency gaxios to v5 (chore: run repo-tools generate #638)
build!: update library to use Node 12 (Add function emulator back in #644)
chore(main): release 3.0.0 (Fix permission of bash script #645)
chore(main): release 3.0.1 (Fix error message, explicitly delete legacy formatted functions #649)
chore(main): release 3.0.2 (Delete Kokoro tests that fail locally #655)
chore(main): release 3.0.3 (Upgrade repo-tools to fix some failing system tests. #657)
chore(main): release 3.0.4 (Update retry samples #677)

google-cla · 2022-11-10T13:17:45Z

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

NimJay · 2022-11-10T15:10:16Z

cla/google GitHub CI check overridden. ✅

snippet-bot · 2022-11-10T21:38:37Z

Here is the summary of changes.

You are about to add 5 region tags.

error-reporting/explicitSetup.js:23, tag error_reporting_setup_nodejs_explicit
error-reporting/express.js:23, tag error_reporting_setup_nodejs_express
error-reporting/implicitSetup.js:23, tag error_reporting_setup_nodejs_implicit
error-reporting/manual.js:23, tag error_reporting_setup_nodejs_manual
error-reporting/quickstart.js:18, tag error_reporting_quickstart

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

Refresh this comment

ace-n

~~Does quickstart.js belong in the google-cloud-node repository?~~ Update: it's fine to keep it here, per internal conversation.

* Add semistandard as a devDependency * Fix storage tests * Fix language tests * Update (some) dependencies * Fix language slackbot tests

* Fix docs-samples tests, round 1 * Fix circle.yml * Add RUN_ALL_BUILDS flag * More container builder bugfixes * Tweak env vars + remove manual proxy install * Env vars in bashrc don't evaluate dynamically, so avoid them * Add semicolons for command ordering * Add appengine/static-files test to circle.yaml * Fix failing container builder tests * Address comments

BREAKING CHANGE: This change renames the `Errors` class to `ErrorReporting`.

* Fixing vulnerability reported by snyk That does not cause any harm to this samples package but I want to keep snyk dashboard clean. * require the correct version of the main package * undo version change

* chore: lock files maintenance * chore: lock files maintenance

* chore: lock files maintenance * chore: lock files maintenance * chore: lock files maintenance

* updated CHANGELOG.md * updated package.json * updated samples/package.json

* updated CHANGELOG.md [ci skip] * updated package.json [ci skip] * updated samples/package.json Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Benjamin E. Coe <bencoe@google.com>

This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [mocha](https://mochajs.org/) ([source](https://github.com/mochajs/mocha)) | devDependencies | major | [`^7.0.0` -> `^8.0.0`](https://renovatebot.com/diffs/npm/mocha/7.2.0/8.0.1) | --- ### Release Notes <details> <summary>mochajs/mocha</summary> ### [`v8.0.1`](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md#801--2020-06-10) [Compare Source](https://github.com/mochajs/mocha/compare/v8.0.0...v8.0.1) The obligatory patch after a major. #### 🐛 Fixes - [#4328](https://github.com/mochajs/mocha/issues/4328): Fix `--parallel` when combined with `--watch` ([**@boneskull**](https://github.com/boneskull)) ### [`v8.0.0`](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md#800--2020-06-10) [Compare Source](https://github.com/mochajs/mocha/compare/v7.2.0...v8.0.0) In this major release, Mocha adds the ability to _run tests in parallel_. Better late than never! Please note the **breaking changes** detailed below. Let's welcome [**@giltayar**](https://github.com/giltayar) and [**@nicojs**](https://github.com/nicojs) to the maintenance team! #### 💥 Breaking Changes - [#4164](https://github.com/mochajs/mocha/issues/4164): **Mocha v8.0.0 now requires Node.js v10.0.0 or newer.** Mocha no longer supports the Node.js v8.x line ("Carbon"), which entered End-of-Life at the end of 2019 ([**@UlisesGascon**](https://github.com/UlisesGascon)) - [#4175](https://github.com/mochajs/mocha/issues/4175): Having been deprecated with a warning since v7.0.0, **`mocha.opts` is no longer supported** ([**@juergba**](https://github.com/juergba)) ✨ **WORKAROUND:** Replace `mocha.opts` with a [configuration file](https://mochajs.org/#configuring-mocha-nodejs). - [#4260](https://github.com/mochajs/mocha/issues/4260): Remove `enableTimeout()` (`this.enableTimeout()`) from the context object ([**@craigtaub**](https://github.com/craigtaub)) ✨ **WORKAROUND:** Replace usage of `this.enableTimeout(false)` in your tests with `this.timeout(0)`. - [#4315](https://github.com/mochajs/mocha/issues/4315): The `spec` option no longer supports a comma-delimited list of files ([**@juergba**](https://github.com/juergba)) ✨ **WORKAROUND**: Use an array instead (e.g., `"spec": "foo.js,bar.js"` becomes `"spec": ["foo.js", "bar.js"]`). - [#4309](https://github.com/mochajs/mocha/issues/4309): Drop support for Node.js v13.x line, which is now End-of-Life ([**@juergba**](https://github.com/juergba)) - [#4282](https://github.com/mochajs/mocha/issues/4282): `--forbid-only` will throw an error even if exclusive tests are avoided via `--grep` or other means ([**@arvidOtt**](https://github.com/arvidOtt)) - [#4223](https://github.com/mochajs/mocha/issues/4223): The context object's `skip()` (`this.skip()`) in a "before all" (`before()`) hook will no longer execute subsequent sibling hooks, in addition to hooks in child suites ([**@juergba**](https://github.com/juergba)) - [#4178](https://github.com/mochajs/mocha/issues/4178): Remove previously soft-deprecated APIs ([**@wnghdcjfe**](https://github.com/wnghdcjfe)): - `Mocha.prototype.ignoreLeaks()` - `Mocha.prototype.useColors()` - `Mocha.prototype.useInlineDiffs()` - `Mocha.prototype.hideDiff()` #### 🎉 Enhancements - [#4245](https://github.com/mochajs/mocha/issues/4245): Add ability to run tests in parallel for Node.js (see [docs](https://mochajs.org/#parallel-tests)) ([**@boneskull**](https://github.com/boneskull)) ❗ See also [#4244](https://github.com/mochajs/mocha/issues/4244); [Root Hook Plugins (docs)](https://mochajs.org/#root-hook-plugins) -- _root hooks must be defined via Root Hook Plugins to work in parallel mode_ - [#4304](https://github.com/mochajs/mocha/issues/4304): `--require` now works with ES modules ([**@JacobLey**](https://github.com/JacobLey)) - [#4299](https://github.com/mochajs/mocha/issues/4299): In some circumstances, Mocha can run ES modules under Node.js v10 -- _use at your own risk!_ ([**@giltayar**](https://github.com/giltayar)) #### 📖 Documentation - [#4246](https://github.com/mochajs/mocha/issues/4246): Add documentation for parallel mode and Root Hook plugins ([**@boneskull**](https://github.com/boneskull)) #### 🐛 Fixes (All bug fixes in Mocha v8.0.0 are also breaking changes, and are listed above) </details> --- ### Renovate configuration 📅 **Schedule**: "after 9am and before 3pm" (UTC). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻️ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/nodejs-error-reporting).

Co-authored-by: Nicole Zhu <69952136+nicoleczhu@users.noreply.github.com> Co-authored-by: Justin Beckwith <justin.beckwith@gmail.com>

🤖 I have created a release \*beep\* \*boop\* --- ### [2.0.1](https://www.github.com/googleapis/nodejs-error-reporting/compare/v2.0.0...v2.0.1) (2021-01-14) ### Bug Fixes * convert AdditionalMessage param into string type CustomMessage ([#535](https://www.github.com/googleapis/nodejs-error-reporting/issues/535)) ([ba7d8b0](https://www.github.com/googleapis/nodejs-error-reporting/commit/ba7d8b01b6351354c88a675bfe55910e7a2c0eff)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

🤖 I have created a release \*beep\* \*boop\* --- ### [2.0.2](https://www.github.com/googleapis/nodejs-error-reporting/compare/v2.0.1...v2.0.2) (2021-05-12) ### Miscellaneous Chores * release 2.0.2 ([#570](https://www.github.com/googleapis/nodejs-error-reporting/issues/570)) ([7733375](https://www.github.com/googleapis/nodejs-error-reporting/commit/7733375a7686e51b4b6bf96aa0f8999a4d012b36)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

🤖 I have created a release \*beep\* \*boop\* --- ### [2.0.3](https://www.github.com/googleapis/nodejs-error-reporting/compare/v2.0.2...v2.0.3) (2021-08-26) ### Bug Fixes * **build:** migrate to using main branch ([#593](https://www.github.com/googleapis/nodejs-error-reporting/issues/593)) ([5e37253](https://www.github.com/googleapis/nodejs-error-reporting/commit/5e37253b82f148719afd534aa48d7643810d33b5)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

🤖 I have created a release \*beep\* \*boop\* --- ### [2.0.4](https://www.github.com/googleapis/nodejs-error-reporting/compare/v2.0.3...v2.0.4) (2021-10-04) ### Bug Fixes * export ErrorMessage as part of the package ([#604](https://www.github.com/googleapis/nodejs-error-reporting/issues/604)) ([eeb0218](https://www.github.com/googleapis/nodejs-error-reporting/commit/eeb02180691260960438b1fbb42425843aa2d677)), closes [#584](https://www.github.com/googleapis/nodejs-error-reporting/issues/584) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

* build!: Update library to use Node 12 Co-authored-by: Summer Ji <summerji@google.com> Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

product-auto-label bot added the samples Issues that are directly related to samples. label Nov 10, 2022

kweinmeister marked this pull request as ready for review November 10, 2022 21:38

kweinmeister requested review from a team as code owners November 10, 2022 21:38

ace-n suggested changes Nov 10, 2022

View reviewed changes

jmdobry and others added 22 commits November 10, 2022 16:22

Error reporting (#408)

7ec9157

Update dependencies + fix a few tests (#448)

08ee4ba

* Add semistandard as a devDependency * Fix storage tests * Fix language tests * Update (some) dependencies * Fix language slackbot tests

Build updates. (#462)

4b64fe8

Complete code migration (#18)

2fbaff8

chore: fix the samples tests (#39)

f763bb7

BREAKING CHANGE: This change renames the `Errors` class to `ErrorReporting`.

Fixing vulnerability reported by snyk (#51)

6975b40

* Fixing vulnerability reported by snyk That does not cause any harm to this samples package but I want to keep snyk dashboard clean. * require the correct version of the main package * undo version change

chore: update README to reflect the new API (#76)

7fcddf0

chore: release v0.4.0 (#109)

bdf66a9

chore: lock files maintenance (#114)

c4bea37

* chore: lock files maintenance * chore: lock files maintenance

chore: lock files maintenance (#118)

72240d7

* chore: lock files maintenance * chore: lock files maintenance

chore: update package locks (#122)

92ffdbb

chore: upgrade to gts 0.6.0 (#123)

26391d7

chore: lock files maintenance (#131)

d516b8c

* chore: lock files maintenance * chore: lock files maintenance * chore: lock files maintenance

fix: upgrade to the latest @google-cloud/common (#128)

303d4b4

fix: update gts and typescript (#143)

0af32c1

chore: update sample lockfiles (#147)

44e84d5

refactor: drop repo-tool as an exec wrapper (#148)

78fcd07

bump version to 0.5.0 (#149)

f78a7ba

chore(deps): update dependency sinon to v5.1.1 (#151)

5f2c1b2

fix(deps): update dependency yargs to v12 (#154)

5c9380b

chore(deps): update dependency sinon to v6 (#153)

1189fed

JustinBeckwith and others added 26 commits November 10, 2022 16:22

docs(samples): modernize samples, sample tests (#346)

384169b

chore: release 1.0.0 (#360)

41fe5de

* updated CHANGELOG.md * updated package.json * updated samples/package.json

chore: release 1.1.0 (#378)

114da2f

* updated CHANGELOG.md * updated package.json * updated samples/package.json

chore: release 1.1.1 (#382)

26cf52c

chore: update license headers

8e6ee5c

chore: release 1.1.3 (#414)

5060059

refactor: use explicit mocha imports (#425)

b92df77

chore(deps): update dependency mocha to v7 (#428)

3db596a

build!: update to latest version of gts and typescript (#467)

72b8ade

chore: release 2.0.0 (#469)

306449e

* updated CHANGELOG.md [ci skip] * updated package.json [ci skip] * updated samples/package.json Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Benjamin E. Coe <bencoe@google.com>

chore(deps): update dependency gaxios to v4 (#524)

234a836

Co-authored-by: Nicole Zhu <69952136+nicoleczhu@users.noreply.github.com> Co-authored-by: Justin Beckwith <justin.beckwith@gmail.com>

chore(main): release 2.0.5 (#636)

5fba9bd

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

chore(deps): update dependency gaxios to v5 (#638)

dd03ca3

build!: update library to use Node 12 (#644)

5114792

* build!: Update library to use Node 12 Co-authored-by: Summer Ji <summerji@google.com> Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

chore(main): release 3.0.0 (#645)

f714636

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

chore(main): release 3.0.1 (#649)

4b1a060

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

chore(main): release 3.0.2 (#655)

bcaa858

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

chore(main): release 3.0.3 (#657)

7635b48

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

chore(main): release 3.0.4 (#677)

3505b5e

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

docs: Add error-reporting samples

77ba53f

ace-n force-pushed the nodejs-error-reporting-migration branch from 4ed30db to 77ba53f Compare November 11, 2022 00:22

ace-n approved these changes Nov 11, 2022

View reviewed changes

ace-n merged commit 6511cf0 into main Nov 11, 2022

ace-n deleted the nodejs-error-reporting-migration branch November 11, 2022 00:41

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

migrate code from googleapis/nodejs-error-reporting #2821

migrate code from googleapis/nodejs-error-reporting #2821

kweinmeister commented Nov 10, 2022

google-cla bot commented Nov 10, 2022

NimJay commented Nov 10, 2022

snippet-bot bot commented Nov 10, 2022 •

edited

Loading

ace-n left a comment •

edited

Loading

migrate code from googleapis/nodejs-error-reporting #2821

migrate code from googleapis/nodejs-error-reporting #2821

Conversation

kweinmeister commented Nov 10, 2022

google-cla bot commented Nov 10, 2022

NimJay commented Nov 10, 2022

snippet-bot bot commented Nov 10, 2022 • edited Loading

ace-n left a comment • edited Loading

Choose a reason for hiding this comment

snippet-bot bot commented Nov 10, 2022 •

edited

Loading

ace-n left a comment •

edited

Loading