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

Add syntax Api to demo #34

Merged
merged 4 commits into from
Feb 8, 2019

Conversation

shranith
Copy link
Contributor

@shranith shranith commented Feb 7, 2019

Added new Syntax API Feature to the NLU Demo.

germanattanasio
germanattanasio previously approved these changes Feb 7, 2019
@germanattanasio germanattanasio dismissed their stale review February 7, 2019 23:18

Travis is failing

Copy link
Contributor

@germanattanasio germanattanasio left a comment

Choose a reason for hiding this comment

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

/home/travis/build/watson-developer-cloud/natural-language-understanding-nodejs/views/utils/request.js
  13:7   error  Unnecessarily quoted property 'tokens' found          quote-props
  14:9   error  Unnecessarily quoted property 'lemma' found           quote-props
  14:17  error  Missing space before value for key 'lemma'            key-spacing
  15:9   error  Unnecessarily quoted property 'part_of_speech' found  quote-props
  17:7   error  Unnecessarily quoted property 'sentences' found       quote-props
  18:6   error  Missing trailing comma                                comma-dangle

@shranith
Copy link
Contributor Author

shranith commented Feb 7, 2019

@germanattanasio The issues are fixed.

@germanattanasio germanattanasio merged commit 3d62c5f into watson-developer-cloud:master Feb 8, 2019
@watson-github-bot
Copy link
Member

🎉 This PR is included in version 0.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants