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.

Sign up for GitHub

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

Jump to bottom

samples: migrate code from googleapis/nodejs-dlp #3035

Merged
merged 176 commits into from
Feb 16, 2023
Merged

samples: migrate code from googleapis/nodejs-dlp #3035

merged 176 commits into from
Feb 16, 2023

Conversation

Shabirmean
Copy link
Member

Migrating samples from googleapis/nodejs-dlp into nodejs-docs-samples/dlp

Ace Nassri and others added 30 commits March 9, 2017 15:41
DLP samples (#322)
1900ae8 
* First draft of DLP samples

* Fix DLP tests

* Add README

* Fix README bugs
Travis: fix failing tests + update dependencies (#335)
7caf7de 
* Make unify script recursive + clean up dependency conflicts

* Restore travis.yml

* Delete outdated text detection sample that duplicates detect.js

* Fix failing KMS + vision tests by updating dependencies

* Fix video tests using a bad cwd

* Upgrade monitoring dependency + skip flaky monitoring tests

* Fix DLP tests having wrong cwd

* Fix failing vision test

* Fix datastore tests

* Update broken dependency

* Update possibly broken compute engine dependency

* Fix typos

* Disable Node 4 testing

* Revert deletion of outdated sample - @gguuss says we still use this.

This reverts commit b7259c8.

* Update dependency
@jmdobry
Cleanup App Engine samples and re-work tests. (#354)
242ead6
@jmdobry
Upgrade to repo tools v1.4.7 (#370)
52b9694
@jmdobry
Upgrade repo tools and regenerate READMEs. (#384)
2cb97b4
@jmdobry
Upgrade repo tools and regenerate READMEs.
f00874b
Add + run dependency updating (bash) script (#401)
13babeb 
* Add + run dependency updating script

* Address comments

Change-Id: I67af9d3ab9e461b579dbaee92274b6163d73c23e

* Run dependency script again

Change-Id: Icbec4acb2cc6bcfa40e0a3a705c5a1059d64efa5

* Update license

Change-Id: Ic1dd0a1bd34356e415bdbf005b81a71a8d2695c2

* Update (more) dependencies

Change-Id: Idaed95b9cfe486797fa75946b6f55e7e702217b8
Move DLP from REST to gRPC + client libraries (#413)
92b4b4f 
* Move DLP from REST to gRPC + client libraries

* Update package.json

* Address comments
Update + autogen README (#414)
d3970d5
Add DLP image redaction sample (Take 2) (#441)
dc56d22 
* Add DLP image redaction sample

* Fix lint
Add DLP quickstart + fix incorrect comments (#442)
86902b2 
* Fix typo in comments

* Add DLP quickstart

* Address comments
Update dependencies + fix a few tests (#448)
5e575f4 
* Add semistandard as a devDependency

* Fix storage tests

* Fix language tests

* Update (some) dependencies

* Fix language slackbot tests
@jmdobry
Build updates. (#462)
87645e6
Update dependencies (#468)
091d519 
* 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
Add DLP samples (BigQuery, DeID, RiskAnalysis) (#474)
030e0cf 
* Add BigQuery samples + a few minor tweaks

* Update comments + fix failing test

* Sync w/codegen changes

* Add DeID samples

* Add DeID tests + remove infoTypes from DeID samples

* Remove unused option

* Add risk analysis samples

* Update README

* Add region tags + fix comment
@lukesneeringer
Repository Migration (#2)
fa8ecdc
@lukesneeringer
Fix three-hour typo (#4)
d92954f
@lukesneeringer
Update sample dependencies (#5)
520281b
@jmdobry @lukesneeringer
Fix docs and regenerate scaffolding. (#7)
06f4904
@stephenplusplus
Test on Node 9. (#8)
9ee9326 
* Test on Node 9.

* Lint.
@greenkeeper @stephenplusplus
Update dependencies to enable Greenkeeper 🌴 (#13)
ae2b575 
* chore(package): update dependencies

* Prettier.
@jmdobry
Clean up DLP region tags. (#26)
23a238e
@alexander-fenster
GAPIC v2 code, release 0.3.0 (#23)
76f2778 
* generated GAPIC v2 code

* lint and prettier

* WIP: update samples

* Upgrade samples + tests

* Fix lint errors + add dateshiftconfig sample

* Remove extraneous triggers field

* Add descriptions to triggers + add missing result file

* s/projectId/callingProject/ + remove project id from deleteJob

* Add templateId + displayName to templates.js

* Remove includeQuote + use dataProjectId

* IMPORTANT: properly process multiple buckets

* IMPORTANT: properly process multiple buckets, pt 2

* Minor tweaks to deid

* Remove inspectconfig-specific print statements

* (Almost-)final draft of Node.js DLP samples

* Regenerate README

* Fix nits

* Move require statements to the top of functions

* Nit: fix comment

* Fix incorrect limit syntax in quickstart

* Clarify that filter parameter is not entirely optional

* Add missing README sections

* Only ack pubsub messages with the desired contents

* Update redact images

* Missed a spot :(

* regenerated v2 GAPIC code, fixed samples and system tests

* get rid of unused response variables

* unskipping some samples test that actually work

* fixing bigquery tests

* Finishing touches - add delay to DlpJob retrieval + make DlpJob tests parallel

* prettier
@alexander-fenster @stephenplusplus
Fix sample tests and system tests (#29)
6b5e881
@jmdobry @alexander-fenster
Upgrade repo-tools and regenerate scaffolding. (#30)
2401004
@alexander-fenster
Fix invalid region code (#32)
af00bc0
@alexander-fenster
tests: fix risk samples tests (#33)
ac44772 
Anonymity ranges can be multi-digit.
@alexander-fenster @callmehiphop
chore: make samples depend on the current version (#34)
0833cd7
Fix region tags (#36)
392c146 
* Fix region tags

* Update risk.js

* Fix region tags
@alexander-fenster
chore: bump version to 0.4.0 (#39)
4525c4b
sofisl and others added 14 commits May 19, 2022 17:28
@sofisl
build!: update library to use Node 12 (#713)
676edb1 
* build!: Update library to use Node 12
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
@release-please
chore(main): release 4.0.0 (#714)
de93e7b 
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>
@renovate-bot
fix(deps): update dependency @google-cloud/pubsub to v3 (#715)
15c877f
@release-please
chore(main): release 4.0.1 (#717)
5d4129b 
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>
Co-authored-by: [email protected] <[email protected]>
@sofisl
build: fix sample test (#721)
e3b94b7
@sofisl
build: increase retries for dlp test (#723)
7e0365d
@release-please @gcf-owl-bot
chore(main): release 4.0.2 (#725)
a849bd9 
* chore(main): release 4.0.2

* 🦉 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>
@release-please @gcf-owl-bot
chore(main): release 4.1.0 (#729)
9ed7e38 
* chore(main): release 4.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>
@alexander-fenster @gcf-owl-bot
test: fix sample test for listJobs (#734)
5854387 
* test: fix sample test for listJobs

* test: fix sample test for listJobs

* 🦉 Updates from OwlBot post-processor

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

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
@release-please @gcf-owl-bot
chore(main): release 4.1.1 (#737)
bedcd1a 
* chore(main): release 4.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>
@renovate-bot @alexander-fenster
chore(deps): update dependency uuid to v9 (#744)
0174bc1 
* chore(deps): update dependency uuid to v9

* test: fix samples test

Co-authored-by: Alexander Fenster <[email protected]>
@release-please
chore(main): release 4.2.0 (#745)
7d1bfdc 
🤖 I have created a release *beep* *boop*
---


## [4.2.0](googleapis/nodejs-dlp@v4.1.1...v4.2.0) (2022-09-22)


### Features

* Add Deidentify action ([#742](googleapis/nodejs-dlp#742)) ([27bb912](googleapis/nodejs-dlp@27bb912))


### Bug Fixes

* Do not import the whole google-gax from proto JS ([#1553](https://github.com/googleapis/nodejs-dlp/issues/1553)) ([#741](googleapis/nodejs-dlp#741)) ([655d6af](googleapis/nodejs-dlp@655d6af))
* Preserve default values in x-goog-request-params header ([#746](googleapis/nodejs-dlp#746)) ([7c53b9f](googleapis/nodejs-dlp@7c53b9f))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
@release-please
chore(main): release 4.3.0 (#778)
1793f7c 
🤖 I have created a release *beep* *boop*
---


## [4.3.0](https://togithub.com/googleapis/nodejs-dlp/compare/v4.2.0...v4.3.0) (2023-01-06)


### Features

* ExcludeByHotword added as an ExclusionRule type, NEW_ZEALAND added as a LocationCategory value ([5b54b2e](https://togithub.com/googleapis/nodejs-dlp/commit/5b54b2e9c63acee3022089d9fb94d8b1907c1eb2))


### Bug Fixes

* Deprecate extra field to avoid confusion ([#777](https://togithub.com/googleapis/nodejs-dlp/issues/777)) ([f6a7ebd](https://togithub.com/googleapis/nodejs-dlp/commit/f6a7ebde9f78440600ac178a568e3fe79ccfadc2))
* **deps:** Use google-gax v3.5.2 ([#781](https://togithub.com/googleapis/nodejs-dlp/issues/781)) ([3601ed8](https://togithub.com/googleapis/nodejs-dlp/commit/3601ed84fa97958ca52bdc8d5620317f1a4a1de5))
* Regenerated protos JS and TS definitions ([#784](https://togithub.com/googleapis/nodejs-dlp/issues/784)) ([fa109f0](https://togithub.com/googleapis/nodejs-dlp/commit/fa109f05c3a29473a330e96b20e89480fb89496e))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
@Shabirmean
Merge remote-tracking branch 'migration/main' into nodejs-dlp-migration
a6fd652
@Shabirmean Shabirmean requested a review from a team as a code owner February 16, 2023 01:20
@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Feb 16, 2023
@google-cla
Copy link

google-cla bot commented Feb 16, 2023

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.

@snippet-bot
Copy link

snippet-bot bot commented Feb 16, 2023 •
edited
Loading

Here is the summary of changes.

You are about to add 26 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

@Shabirmean Shabirmean requested a review from a team as a code owner February 16, 2023 01:25
@Shabirmean Shabirmean changed the title [WIP] samples: migrate code from googleapis/nodejs-dlp samples: migrate code from googleapis/nodejs-dlp Feb 16, 2023
@Shabirmean
Copy link
Member Author

Shabirmean commented Feb 16, 2023 •
edited
Loading

@GoogleCloudPlatform/nodejs-samples-reviewers @GoogleCloudPlatform/torus-dpe
Please see following commits for changes Imade:

@Shabirmean Shabirmean merged commit fd1600d into main Feb 16, 2023
@Shabirmean Shabirmean deleted the nodejs-dlp-migration branch February 16, 2023 19:51
@Shabirmean Shabirmean mentioned this pull request Feb 17, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kweinmeister kweinmeister kweinmeister approved these changes

Assignees
No one assigned
Labels
samples Issues that are directly related to samples.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.