-
Notifications
You must be signed in to change notification settings - Fork 24
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
fix: make plugin parameters work properly #574
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -79,10 +79,14 @@ export class Generator { | |||
const parameters = parameter.split(','); | |||
for (let param of parameters) { | |||
// remove double quote | |||
param = param.substring(1, param.length - 1); | |||
if (param[0] === '"' && param[param.length - 1] === '"') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is parm.lenght guaranteed to be >=1?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does not matter, it won't crash in JavaScript (will return undefined
).
🤖 I have created a release \*beep\* \*boop\* --- ### [1.0.3](https://www.github.com/googleapis/gapic-generator-typescript/compare/v1.0.2...v1.0.3) (2020-06-24) ### Bug Fixes * make plugin parameters work properly ([#574](https://www.github.com/googleapis/gapic-generator-typescript/issues/574)) ([2d77494](https://www.github.com/googleapis/gapic-generator-typescript/commit/2d774943913fbcc096b39d0786fe603994565ff9)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please).
This version of the generator includes a fix to make plugin parameters work properly: googleapis/gapic-generator-typescript#574 PiperOrigin-RevId: 318086414
This version of the generator includes a fix to make plugin parameters work properly: googleapis/gapic-generator-typescript#574 PiperOrigin-RevId: 318086414 Source-Author: Google APIs <[email protected]> Source-Date: Wed Jun 24 10:05:15 2020 -0700 Source-Repo: googleapis/googleapis Source-Sha: cbf322455cc960260c8c41d9cecaa7c4c3576ce9 Source-Link: googleapis/googleapis@cbf3224
This version of the generator includes a fix to make plugin parameters work properly: googleapis/gapic-generator-typescript#574 PiperOrigin-RevId: 318086414 Source-Author: Google APIs <[email protected]> Source-Date: Wed Jun 24 10:05:15 2020 -0700 Source-Repo: googleapis/googleapis Source-Sha: cbf322455cc960260c8c41d9cecaa7c4c3576ce9 Source-Link: googleapis/googleapis@cbf3224
This version of the generator includes a fix to make plugin parameters work properly: googleapis/gapic-generator-typescript#574 PiperOrigin-RevId: 318086414 Source-Author: Google APIs <[email protected]> Source-Date: Wed Jun 24 10:05:15 2020 -0700 Source-Repo: googleapis/googleapis Source-Sha: cbf322455cc960260c8c41d9cecaa7c4c3576ce9 Source-Link: googleapis/googleapis@cbf3224
Handling extra plugin parameters properly.
Also, adding a debug print to simplify transitioning from Docker containers to Bazel builds.