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

[Merged by Bors] - fix: dialogflow cx (PL-152) #362

Closed

Conversation

DecathectZero
Copy link
Member

Fixes or implements PL-152

Brief description. What is this change?

FREE

@sonarcloud
Copy link

sonarcloud bot commented Sep 5, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@@ -57,6 +57,7 @@ export interface JSONEntity {
startPos: number;
endPos: number;
entity: string;
key: string;
Copy link
Contributor

Choose a reason for hiding this comment

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

This might need a more descriptive variable name or at least a comment with this form

/**
 * This key is responsible for so-and-so
 */

It's kinda hard to tell what this is and why you need it.

@DecathectZero
Copy link
Member Author

bors r+

bors-vf bot pushed a commit that referenced this pull request Sep 5, 2022
<!-- You can erase any parts of this template not applicable to your Pull Request. -->

**Fixes or implements PL-152**

### Brief description. What is this change?
FREE


Co-authored-by: Tyler Han <tylerhan97@gmail.com>
@bors-vf
Copy link

bors-vf bot commented Sep 5, 2022

Pull request successfully merged into master.

Build succeeded:

@bors-vf bors-vf bot changed the title fix: dialogflow cx (PL-152) [Merged by Bors] - fix: dialogflow cx (PL-152) Sep 5, 2022
@bors-vf bors-vf bot closed this Sep 5, 2022
@bors-vf bors-vf bot deleted the tyler/add-dialogflow-platform-type/CT-000 branch September 5, 2022 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants