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

Dialogflow to gcp #1181

Merged
merged 12 commits into from
Aug 28, 2018
Merged

Dialogflow to gcp #1181

merged 12 commits into from
Aug 28, 2018

Conversation

nnegrey
Copy link
Contributor

@nnegrey nnegrey commented Aug 14, 2018

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 14, 2018
@nnegrey nnegrey requested a review from kurtisvg August 27, 2018 18:18
Copy link
Contributor

@kurtisvg kurtisvg left a comment

Choose a reason for hiding this comment

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

s/Google Inc/Google LLC/

@@ -0,0 +1,175 @@
/*
* Copyright 2018 Google Inc.
Copy link
Contributor

Choose a reason for hiding this comment

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

LLC

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm getting a style issue with LLC, is it supposed to be LLC.?

Copy link
Contributor

Choose a reason for hiding this comment

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

No, but you need to update the parent's in the pom.xml so it's using the updated checkstyle (sorry I missed this in the initial review):

  <!--
    The parent pom defines common style checks and testing strategies for our samples.
    Removing or replacing it should not affect the execution of the samples in anyway.
  -->
  <parent>
    <groupId>com.google.cloud.samples</groupId>
    <artifactId>shared-configuration</artifactId>
    <version>1.0.9</version>
  </parent>

@@ -0,0 +1,134 @@
/*
* Copyright 2018 Google Inc.
Copy link
Contributor

Choose a reason for hiding this comment

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

Also LLC

@nnegrey nnegrey added kokoro:run Add this label to force Kokoro to re-run the tests. and removed kokoro:run Add this label to force Kokoro to re-run the tests. labels Aug 27, 2018
@nnegrey
Copy link
Contributor Author

nnegrey commented Aug 27, 2018

Not sure what this error is: Caused by: io.grpc.StatusRuntimeException: INVALID_ARGUMENT: Current user is null gonna try running again. Haven't seen it before.

kurtisvg
kurtisvg previously approved these changes Aug 27, 2018
Copy link
Contributor

@kurtisvg kurtisvg left a comment

Choose a reason for hiding this comment

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

LGTM

@kurtisvg kurtisvg added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 27, 2018
@nnegrey nnegrey changed the title [DO_NOT_MERGE] Dialogflow to gcp Dialogflow to gcp Aug 28, 2018
@nnegrey nnegrey removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 28, 2018
@nnegrey
Copy link
Contributor Author

nnegrey commented Aug 28, 2018

Updated some more region tags. PTAL
Should be good to go now.

@nnegrey nnegrey merged commit eb8f5d5 into master Aug 28, 2018
@nnegrey nnegrey deleted the dialogflow-to-gcp branch August 28, 2018 23:04
Shabirmean pushed a commit that referenced this pull request Nov 14, 2022
* Clean copy of Dialogflow samples, need to update pom.xml to work with java-docs-samples repo

* Update files to match GCP Repo's style guides

* Update argument parsing

* Update Inc. to LLC

* Update pom.xml

* Add missing region tags for sample tracker
Shabirmean pushed a commit that referenced this pull request Nov 15, 2022
* Clean copy of Dialogflow samples, need to update pom.xml to work with java-docs-samples repo

* Update files to match GCP Repo's style guides

* Update argument parsing

* Update Inc. to LLC

* Update pom.xml

* Add missing region tags for sample tracker
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