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

Switch CloudSQL sample to 2nd gen. Added unit tests. #153

Merged
merged 2 commits into from
Jul 21, 2016
Merged

Switch CloudSQL sample to 2nd gen. Added unit tests. #153

merged 2 commits into from
Jul 21, 2016

Conversation

jmdobry
Copy link
Member

@jmdobry jmdobry commented Jul 19, 2016

No description provided.

@codecov-io
Copy link

codecov-io commented Jul 19, 2016 •
edited
Loading

Current coverage is 84.48%

Merging #153 into master will not change coverage

@@             master       #153   diff @@
==========================================
  Files            45         45          
  Lines          2082       2082          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           1759       1759          
  Misses          323        323          
  Partials          0          0

Powered by Codecov. Last updated by ec9ae81...f221b41

@jmdobry jmdobry changed the title Switch CloudSQL sample to 2nd gen. Add units tests. Switch CloudSQL sample to 2nd gen. Added unit tests. Jul 19, 2016
@jmdobry jmdobry assigned jsimonweb and theacodes and unassigned theacodes Jul 20, 2016
jsimonweb
jsimonweb reviewed Jul 20, 2016
View reviewed changes
appengine/cloudsql/createTables.js
');';

/**
* Create a the "visits" table.
Copy link
Contributor

Choose a reason for hiding this comment

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

~ Create the "visits" table

@jsimonweb
Copy link
Contributor

One minor comment update suggestion, otherwise LGTM

@jmdobry jmdobry merged commit 9c5f942 into master Jul 21, 2016
@jmdobry jmdobry deleted the sql branch July 21, 2016 05:40
NimJay pushed a commit that referenced this pull request Nov 9, 2022
@release-please
chore: release 1.10.0 (#153)
2324d29 
🤖 I have created a release \*beep\* \*boop\* 
---
## [1.10.0](https://www.github.com/googleapis/nodejs-containeranalysis/compare/v1.9.0...v1.10.0) (2020-03-06)


### Features

* deferred client initialization ([#152](https://www.github.com/googleapis/nodejs-containeranalysis/issues/152)) ([0a01d71](https://www.github.com/googleapis/nodejs-containeranalysis/commit/0a01d71cc9b7f45c78215838d13c506e316d904d))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please).
@NimJay NimJay mentioned this pull request Nov 9, 2022
Closed
pattishin pushed a commit that referenced this pull request Nov 9, 2022
@SurferJeffAtGoogle @gcf-owl-bot @sofisl
chore: migrate to owl bot (#153)
d9a245a 
* chore: migrate to owl bot

* chore: copy files from googleapis-gen fb91803ccef5d7c695139b22788b309e2197856b

* chore: run the post processor

* 🦉 Updates from OwlBot

* fix: help tests pass

* 🦉 Updates from OwlBot

* update Docker image

* 🦉 Updates from OwlBot

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Sofia Leon <sofialeon@google.com>
@pattishin pattishin mentioned this pull request Nov 9, 2022
Closed
pattishin pushed a commit that referenced this pull request Nov 9, 2022
@SurferJeffAtGoogle @gcf-owl-bot @sofisl
chore: migrate to owl bot (#153)
dda23f4 
* chore: migrate to owl bot

* chore: copy files from googleapis-gen fb91803ccef5d7c695139b22788b309e2197856b

* chore: run the post processor

* 🦉 Updates from OwlBot

* fix: help tests pass

* 🦉 Updates from OwlBot

* update Docker image

* 🦉 Updates from OwlBot

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Sofia Leon <sofialeon@google.com>
@pattishin pattishin mentioned this pull request Nov 9, 2022
Closed
pattishin pushed a commit that referenced this pull request Nov 9, 2022
@SurferJeffAtGoogle @gcf-owl-bot @sofisl
chore: migrate to owl bot (#153)
1673277 
* chore: migrate to owl bot

* chore: copy files from googleapis-gen fb91803ccef5d7c695139b22788b309e2197856b

* chore: run the post processor

* 🦉 Updates from OwlBot

* fix: help tests pass

* 🦉 Updates from OwlBot

* update Docker image

* 🦉 Updates from OwlBot

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Sofia Leon <sofialeon@google.com>
@pattishin pattishin mentioned this pull request Nov 9, 2022
Closed
kweinmeister pushed a commit that referenced this pull request Nov 10, 2022
@renovate @JustinBeckwith
chore(deps): update dependency sinon to v6 (#153)
4393153
@kweinmeister kweinmeister mentioned this pull request Nov 10, 2022
Merged
ace-n pushed a commit that referenced this pull request Nov 11, 2022
@renovate
chore(deps): update dependency sinon to v6 (#153)
1189fed
ace-n pushed a commit that referenced this pull request Nov 11, 2022
@renovate
chore(deps): update dependency sinon to v6 (#153)
97746b6
This was referenced Nov 11, 2022
Closed
Merged
Closed
@pattishin pattishin mentioned this pull request Nov 11, 2022
Merged
NimJay pushed a commit that referenced this pull request Nov 11, 2022
@SurferJeffAtGoogle @gcf-owl-bot
@sofisl @NimJay
chore: migrate to owl bot (#153)
881d62e 
* chore: migrate to owl bot

* chore: copy files from googleapis-gen fb91803ccef5d7c695139b22788b309e2197856b

* chore: run the post processor

* 🦉 Updates from OwlBot

* fix: help tests pass

* 🦉 Updates from OwlBot

* update Docker image

* 🦉 Updates from OwlBot

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Sofia Leon <sofialeon@google.com>
ace-n pushed a commit that referenced this pull request Nov 11, 2022
@jkwlui @yoshi-automation
Release @google-cloud/automl v0.2.0
c0ed8fa 
Release @google-cloud/automl v0.2.0

This pull request was generated using releasetool.

#153 automerged by dpebot
@ace-n ace-n mentioned this pull request Nov 11, 2022
Closed
ace-n pushed a commit that referenced this pull request Nov 11, 2022
@release-please
chore: release 2.4.0 (#153)
60f1a34 
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
ace-n pushed a commit that referenced this pull request Nov 11, 2022
@jkwlui @yoshi-automation
Release @google-cloud/automl v0.2.0
003b3cb 
Release @google-cloud/automl v0.2.0

This pull request was generated using releasetool.

#153 automerged by dpebot
ace-n pushed a commit that referenced this pull request Nov 11, 2022
@release-please
chore: release 2.4.0 (#153)
b6ae8c0 
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
@ace-n ace-n mentioned this pull request Nov 11, 2022
Merged
ace-n pushed a commit that referenced this pull request Nov 12, 2022
@release-please
chore: release 2.4.0 (#153)
22fd34e 
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
ace-n pushed a commit that referenced this pull request Nov 14, 2022
@JustinBeckwith @alexander-fenster
Enable no-var in eslint (#153)
ecc2917
@ace-n ace-n mentioned this pull request Nov 14, 2022
Closed
ace-n pushed a commit that referenced this pull request Nov 14, 2022
@jkwlui
Release @google-cloud/automl v0.2.0
2bd0daa 
Release @google-cloud/automl v0.2.0

This pull request was generated using releasetool.

#153 automerged by dpebot
ace-n pushed a commit that referenced this pull request Nov 14, 2022
@release-please
chore: release 2.4.0 (#153)
f6b0551 
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
ace-n pushed a commit that referenced this pull request Nov 15, 2022
@jkwlui
Release @google-cloud/automl v0.2.0
f94d1ab 
Release @google-cloud/automl v0.2.0

This pull request was generated using releasetool.

#153 automerged by dpebot
@ace-n ace-n mentioned this pull request Nov 15, 2022
Merged
ace-n pushed a commit that referenced this pull request Nov 15, 2022
@jkwlui
Release @google-cloud/automl v0.2.0
0a428b7 
Release @google-cloud/automl v0.2.0

This pull request was generated using releasetool.

#153 automerged by dpebot
ace-n pushed a commit that referenced this pull request Nov 16, 2022
@happyhuman @JustinBeckwith
chore: update samples to use GA APIs (#153)
8ae68d6
@ace-n ace-n mentioned this pull request Nov 16, 2022
Merged
ace-n pushed a commit that referenced this pull request Nov 17, 2022
@happyhuman
chore: update samples to use GA APIs (#153)
d6edf16
ace-n pushed a commit that referenced this pull request Nov 17, 2022
@jkwlui
Release @google-cloud/automl v0.2.0
dc5c7db 
Release @google-cloud/automl v0.2.0

This pull request was generated using releasetool.

#153 automerged by dpebot
ace-n pushed a commit that referenced this pull request Nov 17, 2022
@jkwlui
Release @google-cloud/automl v0.2.0
191dedb 
Release @google-cloud/automl v0.2.0

This pull request was generated using releasetool.

#153 automerged by dpebot
unforced pushed a commit that referenced this pull request Nov 17, 2022
@release-please
chore: release 3.1.3 (#154)
ab36206 
🤖 I have created a release \*beep\* \*boop\*
---
### [3.1.3](https://www.github.com/googleapis/nodejs-service-directory/compare/v3.1.2...v3.1.3) (2021-06-23)


### Bug Fixes

* make request optional in all cases ([#153](https://www.github.com/googleapis/nodejs-service-directory/issues/153)) ([3bc4004](https://www.github.com/googleapis/nodejs-service-directory/commit/3bc4004be93fadaf15b84c7aea31eba3450ddd8d))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
unforced pushed a commit that referenced this pull request Nov 17, 2022
@release-please
chore: release 3.1.3 (#154)
4b65fec 
🤖 I have created a release \*beep\* \*boop\*
---
### [3.1.3](https://www.github.com/googleapis/nodejs-service-directory/compare/v3.1.2...v3.1.3) (2021-06-23)


### Bug Fixes

* make request optional in all cases ([#153](https://www.github.com/googleapis/nodejs-service-directory/issues/153)) ([3bc4004](https://www.github.com/googleapis/nodejs-service-directory/commit/3bc4004be93fadaf15b84c7aea31eba3450ddd8d))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
ahrarmonsur pushed a commit that referenced this pull request Nov 17, 2022
@happyhuman @ahrarmonsur
chore: update samples to use GA APIs (#153)
735b709
ahrarmonsur pushed a commit that referenced this pull request Nov 17, 2022
@jkwlui @ahrarmonsur
Release @google-cloud/automl v0.2.0
5df70ec 
Release @google-cloud/automl v0.2.0

This pull request was generated using releasetool.

#153 automerged by dpebot
ahrarmonsur pushed a commit that referenced this pull request Nov 17, 2022
@release-please @ahrarmonsur
chore: release 3.1.3 (#154)
c85679d 
🤖 I have created a release \*beep\* \*boop\*
---
### [3.1.3](https://www.github.com/googleapis/nodejs-service-directory/compare/v3.1.2...v3.1.3) (2021-06-23)


### Bug Fixes

* make request optional in all cases ([#153](https://www.github.com/googleapis/nodejs-service-directory/issues/153)) ([3bc4004](https://www.github.com/googleapis/nodejs-service-directory/commit/3bc4004be93fadaf15b84c7aea31eba3450ddd8d))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
pattishin pushed a commit that referenced this pull request Nov 18, 2022
@release-please
chore: release 1.7.2 (#153)
bde81ef 
🤖 I have created a release \*beep\* \*boop\*
---
### [1.7.2](https://www.github.com/googleapis/nodejs-ai-platform/compare/v1.7.1...v1.7.2) (2021-06-22)


### Bug Fixes

* make request optional in all cases ([#152](https://www.github.com/googleapis/nodejs-ai-platform/issues/152)) ([96b9fba](https://www.github.com/googleapis/nodejs-ai-platform/commit/96b9fba9b66807ac5ea2c5826d5fd3f4e895dc9e))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
@pattishin pattishin mentioned this pull request Nov 18, 2022
Merged
ace-n pushed a commit that referenced this pull request Nov 21, 2022
@JustinBeckwith
Enable no-var in eslint (#153)
48df157
ace-n pushed a commit that referenced this pull request Nov 21, 2022
@JustinBeckwith
Enable no-var in eslint (#153)
20bbf99
pattishin pushed a commit that referenced this pull request Nov 22, 2022
@release-please @pattishin
chore: release 1.7.2 (#153)
a38a3de 
🤖 I have created a release \*beep\* \*boop\*
---
### [1.7.2](https://www.github.com/googleapis/nodejs-ai-platform/compare/v1.7.1...v1.7.2) (2021-06-22)


### Bug Fixes

* make request optional in all cases ([#152](https://www.github.com/googleapis/nodejs-ai-platform/issues/152)) ([96b9fba](https://www.github.com/googleapis/nodejs-ai-platform/commit/96b9fba9b66807ac5ea2c5826d5fd3f4e895dc9e))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
jsimonweb pushed a commit to jsimonweb/nodejs-docs-samples that referenced this pull request Dec 12, 2022
@renovate-bot
chore(deps): update google-github-actions/setup-gcloud action to v0.6…
aa5bfa2 
….2 (GoogleCloudPlatform#153)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@theacodes theacodes

@jsimonweb jsimonweb

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jmdobry @codecov-io @jsimonweb @theacodes