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

migrate code from googleapis/nodejs-automl #2841

Merged
merged 147 commits into from
Nov 17, 2022
Merged

migrate code from googleapis/nodejs-automl #2841

merged 147 commits into from
Nov 17, 2022

Conversation

ace-n
Copy link
Contributor

@ace-n ace-n commented Nov 11, 2022

@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Nov 11, 2022
@google-cla
Copy link

google-cla bot commented Nov 11, 2022

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.

@ace-n
Copy link
Contributor Author

ace-n commented Nov 11, 2022

automl breaks the migration script

In order to get this PR, I had to add an || true to mkdir -p ${WORKDIR}/migrated-source; mv * ${WORKDIR}/migrated-source; mkdir -p ${TARGET_PATH}; mv ${WORKDIR}/migrated-source/* ${TARGET_PATH}


🚨 🚨 DO NOT MERGE unless @chingor13 signs off 🚨 🚨

@ace-n ace-n added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 11, 2022
@ace-n ace-n closed this Nov 11, 2022
@ace-n ace-n reopened this Nov 11, 2022
@ace-n ace-n closed this Nov 11, 2022
@ace-n ace-n deleted the nodejs-automl-migration branch November 11, 2022 23:56
@ace-n ace-n restored the nodejs-automl-migration branch November 14, 2022 22:43
@ace-n ace-n reopened this Nov 14, 2022
@ace-n ace-n removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 14, 2022
@ace-n
Copy link
Contributor Author

ace-n commented Nov 14, 2022

🧙 Reviving this PR - @chingor13 confirmed my || true workaround was OK.

@ace-n ace-n force-pushed the nodejs-automl-migration branch 3 times, most recently from 043a3a0 to b5fe1e6 Compare November 15, 2022 01:04
@ace-n
Copy link
Contributor Author

ace-n commented Nov 15, 2022

Tests were failing (seemingly) due to a missing BQ database.

I just rebased against main in case that fixes anything!

@ace-n ace-n force-pushed the nodejs-automl-migration branch 2 times, most recently from 077f5d2 to 600512e Compare November 15, 2022 23:49
@ace-n ace-n marked this pull request as ready for review November 16, 2022 00:08
@ace-n ace-n requested review from a team as code owners November 16, 2022 00:08
@snippet-bot
Copy link

snippet-bot bot commented Nov 16, 2022

Here is the summary of changes.

You are about to add 120 region tags.

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 ace-n added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 16, 2022
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 16, 2022
release-please bot and others added 26 commits November 16, 2022 16:09
Co-authored-by: Benjamin E. Coe <bencoe@google.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Karl Weinmeister <11586922+kweinmeister@users.noreply.github.com>
* Added retries to automlTablesPredict sample test

* Convert arrow functions
#564)

* Added retries to automlTablesPredict sample test

* Convert arrow functions

* Add retries to another function in predict test
- [ ] Regenerate this pull request now.

PiperOrigin-RevId: 408439482

Source-Link: googleapis/googleapis@b9f6184

Source-Link: https://github.com/googleapis/googleapis-gen/commit/eb888bc214efc7bf43bf4634b470254565a659a5
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZWI4ODhiYzIxNGVmYzdiZjQzYmY0NjM0YjQ3MDI1NDU2NWE2NTlhNSJ9
Fixes http://b/208672646. Test looks good, going to add comment to Java and Python test files to be similar to Node.js test.
[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [csv](https://csv.js.org/) ([source](https://togithub.com/adaltas/node-csv)) | [`^5.1.3` -> `^6.0.0`](https://renovatebot.com/diffs/npm/csv/5.5.3/6.0.5) | [![age](https://badges.renovateapi.com/packages/npm/csv/6.0.5/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/csv/6.0.5/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/csv/6.0.5/compatibility-slim/5.5.3)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/csv/6.0.5/confidence-slim/5.5.3)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>adaltas/node-csv</summary>

### [`v6.0.5`](https://togithub.com/adaltas/node-csv/blob/HEAD/packages/csv/CHANGELOG.md#&#8203;605-httpsgithubcomadaltasnode-csvcomparecsv604csv605-2021-12-29)

[Compare Source](https://togithub.com/adaltas/node-csv/compare/csv@6.0.4...csv@6.0.5)

##### Bug Fixes

-   correct exports in package.json with webpack ([154eafb](https://togithub.com/adaltas/node-csv/commit/154eafbac866eb4499a0d392f8dcd057695c2586))
-   **csv-demo-webpack-ts:** simplify export paths ([8d63a14](https://togithub.com/adaltas/node-csv/commit/8d63a14313bb6b26f13fafb740cc686f1dfaa65f))
-   esm exports in package.json files ([c48fe47](https://togithub.com/adaltas/node-csv/commit/c48fe478ced7560aa078fbc36ec33d6007111e2b)), closes [#&#8203;308](https://togithub.com/adaltas/node-csv/issues/308)

### [`v6.0.4`](https://togithub.com/adaltas/node-csv/blob/HEAD/packages/csv/CHANGELOG.md#&#8203;604-httpsgithubcomadaltasnode-csvcomparecsv603csv604-2021-11-19)

[Compare Source](https://togithub.com/adaltas/node-csv/compare/csv@6.0.3...csv@6.0.4)

##### Bug Fixes

-   **csv-stringify:** node 12 compatibility in flush ([9145b75](https://togithub.com/adaltas/node-csv/commit/9145b75012ec71a0b4152036af2275bf28c460e0))

### [`v6.0.3`](https://togithub.com/adaltas/node-csv/blob/HEAD/packages/csv/CHANGELOG.md#&#8203;603-httpsgithubcomadaltasnode-csvcomparecsv602csv603-2021-11-19)

[Compare Source](https://togithub.com/adaltas/node-csv/compare/csv@6.0.2...csv@6.0.3)

##### Bug Fixes

-   expose browser esm modules ([eb87355](https://togithub.com/adaltas/node-csv/commit/eb873557c65912f065d2581d30a17a96b0bfd2d6))

### [`v6.0.2`](https://togithub.com/adaltas/node-csv/blob/HEAD/packages/csv/CHANGELOG.md#&#8203;602-httpsgithubcomadaltasnode-csvcomparecsv601csv602-2021-11-18)

[Compare Source](https://togithub.com/adaltas/node-csv/compare/csv@6.0.1...csv@6.0.2)

##### Bug Fixes

-   dont insert polyfills in cjs [#&#8203;303](https://togithub.com/adaltas/node-csv/issues/303) ([9baf334](https://togithub.com/adaltas/node-csv/commit/9baf334044dab90b4a0d096a7e456d0fd5807d5b))

### [`v6.0.1`](https://togithub.com/adaltas/node-csv/blob/HEAD/packages/csv/CHANGELOG.md#&#8203;601-httpsgithubcomadaltasnode-csvcomparecsv600csv601-2021-11-15)

[Compare Source](https://togithub.com/adaltas/node-csv/compare/csv@6.0.0...csv@6.0.1)

##### Bug Fixes

-   remove samples from publicatgion ([12c221d](https://togithub.com/adaltas/node-csv/commit/12c221dc37add26f094e3bb7f94b50ee06ff5be6))

### [`v6.0.0`](https://togithub.com/adaltas/node-csv/blob/HEAD/packages/csv/CHANGELOG.md#&#8203;600-httpsgithubcomadaltasnode-csvcomparecsv553csv600-2021-11-15)

[Compare Source](https://togithub.com/adaltas/node-csv/compare/v5.5.3...csv@6.0.0)

##### Bug Fixes

-   **csv-generate:** finish called twice in node 16 ([3decdf1](https://togithub.com/adaltas/node-csv/commit/3decdf169ce3b8e0c5cadd257816c346c8e4d3fa))
-   **csv:** export csv_sync ([1353284](https://togithub.com/adaltas/node-csv/commit/1353284aa02bb9f4f727d2653e398a869eebe20d))
-   export original lib esm modules ([be25349](https://togithub.com/adaltas/node-csv/commit/be2534928ba21156e9cde1e15d2e8593d62ffe71))
-   fallback to setTimeout is setImmediate is undefined ([3d6a2d0](https://togithub.com/adaltas/node-csv/commit/3d6a2d0a655af342f28456b46db7ccfe7ee9d664))
-   refer to esm files in dist ([b780fbd](https://togithub.com/adaltas/node-csv/commit/b780fbd26f5e54494e511eb2e004d3cdedee3593))

##### Features

-   backport support for node 14 ([dbfeb78](https://togithub.com/adaltas/node-csv/commit/dbfeb78f61ed36f02936d63a53345708ca213e45))
-   backward support for node 8 ([496231d](https://togithub.com/adaltas/node-csv/commit/496231dfd838f0a6a72269a5a2390a4c637cef95))
-   esm migration ([b5c0d4b](https://togithub.com/adaltas/node-csv/commit/b5c0d4b191c8b57397808c0922a3f08248506a9f))
-   export ts types in sync ([890bf8d](https://togithub.com/adaltas/node-csv/commit/890bf8d950c18a05cab5e35a461d0847d9425156))
-   replace ts types with typesVersions ([acb41d5](https://togithub.com/adaltas/node-csv/commit/acb41d5031669f2d582e40da1c80f5fd4738fee4))

#### [5.5.2](https://togithub.com/adaltas/node-csv/compare/csv@5.5.1...csv@5.5.2) (2021-08-27)

**Note:** Version bump only for package csv

#### 5.5.1 (2021-08-27)

**Note:** Version bump only for package csv

#### Version 5.5.0

feat: browersify

#### Version 5.4.0

-   csv-generate: 3.3.0
-   csv-parse: 4.15.3
-   stream-transform: 2.0.4
-   csv-stringify: 5.6.2

#### Version 5.3.2

-   package: update project description
-   csv-parse: 4.8.8
-   csv-stringify: 5.3.6

#### Version 5.3.1

-   csv-generate: 3.2.4
-   csv-parse: 4.8.2
-   csv-stringify: 5.3.4

#### Version 5.3.0

-   csv-parse: version 4.8.0

#### Version 5.2.0

-   csv-parse: version 4.7.0
-   package: contributing
-   package: code of conduct

#### Version 5.1.3

-   travis: add node.js 12
-   csv-parse: version 4.4.6

#### Version 5.1.2

-   csv-generate: version 3.2.3
-   csv-parse: version 4.4.5
-   csv-stringify: version 5.3.3
-   stream-transform: version 2.0.1
-   ts: new definition files
-   project: fix license in package.json

#### Version 5.1.1

-   es5: fix modules to wrap es5 dependencies

#### Version 5.1.0

-   csv-generate: version 3.2.0
-   csv-stringify: version 5.1.2
-   csv-parse: version 4.3.0
-   stream-transform: version 1.0.8
-   babel: re-integration

#### Version 5.0.1

-   all: modules written in native JavaScript
-   package: switch to MIT license

#### Version 5.0.0

-   csv-generate: version 3.1.0
-   csv-stringify: version 5.0.0
-   csv-parse: version 4.0.1
-   travis: test with Node.js 8, 10 and 11

#### Version 4.0.0

-   es5: support older version of Node.js using Babel
-   readme: update project website url
-   sync: expose the sync api
-   samples: new pipe_funny script
-   samples: rewrite pipe with comments

#### Version 3.1.0

-   csv-parse  ^2.2.0  →  ^2.4.0
-   package: ignore lock files

#### Version 3.0.2

-   package: generate js file before release

#### Version 3.0.1

-   package: attempt to re-submit NPM package, no ./lib/index.js present

#### Version 3.0.0

-   csv-generate: "^2.0.2"
-   csv-parse: "^2.2.0"
-   stream-transform: "^1.0.2"
-   csv-stringify: "^3.0.0"

</details>

---

### 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.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/nodejs-automl).
* build!: Update library to use Node 12
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
* chore(main): release 3.1.0

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
* chore(main): release 3.1.1

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [uuid](https://togithub.com/uuidjs/uuid) | [`^8.0.0` -> `^9.0.0`](https://renovatebot.com/diffs/npm/uuid/8.3.2/9.0.0) | [![age](https://badges.renovateapi.com/packages/npm/uuid/9.0.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/uuid/9.0.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/uuid/9.0.0/compatibility-slim/8.3.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/uuid/9.0.0/confidence-slim/8.3.2)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>uuidjs/uuid</summary>

### [`v9.0.0`](https://togithub.com/uuidjs/uuid/blob/HEAD/CHANGELOG.md#&#8203;900-httpsgithubcomuuidjsuuidcomparev832v900-2022-09-05)

[Compare Source](https://togithub.com/uuidjs/uuid/compare/v8.3.2...v9.0.0)

##### ⚠ BREAKING CHANGES

-   Drop Node.js 10.x support. This library always aims at supporting one EOLed LTS release which by this time now is 12.x which has reached EOL 30 Apr 2022.

-   Remove the minified UMD build from the package.

    Minified code is hard to audit and since this is a widely used library it seems more appropriate nowadays to optimize for auditability than to ship a legacy module format that, at best, serves educational purposes nowadays.

    For production browser use cases, users should be using a bundler. For educational purposes, today's online sandboxes like replit.com offer convenient ways to load npm modules, so the use case for UMD through repos like UNPKG or jsDelivr has largely vanished.

-   Drop IE 11 and Safari 10 support. Drop support for browsers that don't correctly implement const/let and default arguments, and no longer transpile the browser build to ES2015.

    This also removes the fallback on msCrypto instead of the crypto API.

    Browser tests are run in the first supported version of each supported browser and in the latest (as of this commit) version available on Browserstack.

##### Features

-   optimize uuid.v1 by 1.3x uuid.v4 by 4.3x (430%) ([#&#8203;597](https://togithub.com/uuidjs/uuid/issues/597)) ([3a033f6](https://togithub.com/uuidjs/uuid/commit/3a033f6bab6bb3780ece6d645b902548043280bc))
-   remove UMD build ([#&#8203;645](https://togithub.com/uuidjs/uuid/issues/645)) ([e948a0f](https://togithub.com/uuidjs/uuid/commit/e948a0f22bf22f4619b27bd913885e478e20fe6f)), closes [#&#8203;620](https://togithub.com/uuidjs/uuid/issues/620)
-   use native crypto.randomUUID when available ([#&#8203;600](https://togithub.com/uuidjs/uuid/issues/600)) ([c9e076c](https://togithub.com/uuidjs/uuid/commit/c9e076c852edad7e9a06baaa1d148cf4eda6c6c4))

##### Bug Fixes

-   add Jest/jsdom compatibility ([#&#8203;642](https://togithub.com/uuidjs/uuid/issues/642)) ([16f9c46](https://togithub.com/uuidjs/uuid/commit/16f9c469edf46f0786164cdf4dc980743984a6fd))
-   change default export to named function ([#&#8203;545](https://togithub.com/uuidjs/uuid/issues/545)) ([c57bc5a](https://togithub.com/uuidjs/uuid/commit/c57bc5a9a0653273aa639cda9177ce52efabe42a))
-   handle error when parameter is not set in v3 and v5 ([#&#8203;622](https://togithub.com/uuidjs/uuid/issues/622)) ([fcd7388](https://togithub.com/uuidjs/uuid/commit/fcd73881692d9fabb63872576ba28e30ff852091))
-   run npm audit fix ([#&#8203;644](https://togithub.com/uuidjs/uuid/issues/644)) ([04686f5](https://togithub.com/uuidjs/uuid/commit/04686f54c5fed2cfffc1b619f4970c4bb8532353))
-   upgrading from uuid3 broken link ([#&#8203;568](https://togithub.com/uuidjs/uuid/issues/568)) ([1c849da](https://togithub.com/uuidjs/uuid/commit/1c849da6e164259e72e18636726345b13a7eddd6))

##### build

-   drop Node.js 8.x from babel transpile target ([#&#8203;603](https://togithub.com/uuidjs/uuid/issues/603)) ([aa11485](https://togithub.com/uuidjs/uuid/commit/aa114858260402107ec8a1e1a825dea0a259bcb5))

-   drop support for legacy browsers (IE11, Safari 10) ([#&#8203;604](https://togithub.com/uuidjs/uuid/issues/604)) ([0f433e5](https://togithub.com/uuidjs/uuid/commit/0f433e5ec444edacd53016de67db021102f36148))

-   drop node 10.x to upgrade dev dependencies ([#&#8203;653](https://togithub.com/uuidjs/uuid/issues/653)) ([28a5712](https://togithub.com/uuidjs/uuid/commit/28a571283f8abda6b9d85e689f95b7d3ee9e282e)), closes [#&#8203;643](https://togithub.com/uuidjs/uuid/issues/643)

##### [8.3.2](https://togithub.com/uuidjs/uuid/compare/v8.3.1...v8.3.2) (2020-12-08)

##### Bug Fixes

-   lazy load getRandomValues ([#&#8203;537](https://togithub.com/uuidjs/uuid/issues/537)) ([16c8f6d](https://togithub.com/uuidjs/uuid/commit/16c8f6df2f6b09b4d6235602d6a591188320a82e)), closes [#&#8203;536](https://togithub.com/uuidjs/uuid/issues/536)

##### [8.3.1](https://togithub.com/uuidjs/uuid/compare/v8.3.0...v8.3.1) (2020-10-04)

##### Bug Fixes

-   support expo>=39.0.0 ([#&#8203;515](https://togithub.com/uuidjs/uuid/issues/515)) ([c65a0f3](https://togithub.com/uuidjs/uuid/commit/c65a0f3fa73b901959d638d1e3591dfacdbed867)), closes [#&#8203;375](https://togithub.com/uuidjs/uuid/issues/375)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 9am and before 3pm" (UTC), Automerge - At any time (no schedule defined).

🚦 **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.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/nodejs-automl).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4xOTEuMCIsInVwZGF0ZWRJblZlciI6IjMyLjE5MS4wIn0=-->
🤖 I have created a release *beep* *boop*
---


## [3.1.2](https://togithub.com/googleapis/nodejs-automl/compare/v3.1.1...v3.1.2) (2022-09-22)


### Bug Fixes

* Preserve default values in x-goog-request-params header ([#634](https://togithub.com/googleapis/nodejs-automl/issues/634)) ([1949bf9](https://togithub.com/googleapis/nodejs-automl/commit/1949bf9a26b63b4eefd13f5c6164691db46a0df2))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
@ace-n ace-n enabled auto-merge (rebase) November 17, 2022 00:13
@ace-n ace-n merged commit 27460d3 into main Nov 17, 2022
@ace-n ace-n deleted the nodejs-automl-migration branch November 17, 2022 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.