Skip to content

feat!: run the generator#1709

Merged
JustinBeckwith merged 2 commits into
masterfrom
ng
May 13, 2019
Merged

feat!: run the generator#1709
JustinBeckwith merged 2 commits into
masterfrom
ng

Conversation

@JustinBeckwith

@JustinBeckwith JustinBeckwith commented May 13, 2019

Copy link
Copy Markdown
Contributor

BREAKING CHANGE: This release has a few breaking changes.

  • Node.js 8.x is now the minimum required version
  • The healthcare/v1alpha API has been removed

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 13, 2019
@JustinBeckwith JustinBeckwith changed the title feat: run the generator feat!: run the generator May 13, 2019
@JustinBeckwith JustinBeckwith requested review from bcoe and jkwlui May 13, 2019 23:32

@bcoe bcoe left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM; although this is very much a rubber stamp.

Comment thread package.json
"nock": "^10.0.6",
"nunjucks": "^3.2.0",
"nyc": "^14.1.1",
"open": "^6.3.0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

curious why we moved from opn to open.

Comment thread package.json
"google-auth-library": "^3.0.0",
"googleapis-common": "^0.7.0"
"google-auth-library": "^4.0.0",
"googleapis-common": "^2.0.0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this is the the most important bit right?

@@ -29,7 +29,7 @@
"webpack": "webpack"
},
"dependencies": {
"googleapis-common": "^0.7.0"
"googleapis-common": "^2.0.0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍 good to see this picked up in each package.

@JustinBeckwith JustinBeckwith merged commit 6817c39 into master May 13, 2019
@JustinBeckwith JustinBeckwith deleted the ng branch December 13, 2020 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants