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

Use new intent parser message structure. #1317

Merged
merged 1 commit into from
Sep 6, 2018

Conversation

mrstegeman
Copy link
Contributor

@codecov-io
Copy link

codecov-io commented Aug 29, 2018

Codecov Report

Merging #1317 into master will increase coverage by 0.07%.
The diff coverage is 68.96%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1317      +/-   ##
=========================================
+ Coverage   75.12%   75.2%   +0.07%     
=========================================
  Files         123     123              
  Lines        5974    5989      +15     
  Branches      839     842       +3     
=========================================
+ Hits         4488    4504      +16     
  Misses       1312    1312              
+ Partials      174     173       -1
Impacted Files Coverage Δ
src/models/intentparser.js 79.54% <68.96%> (-6.67%) ⬇️
src/models/things.js 98.71% <0%> (+1.28%) ⬆️
src/models/thing.js 68.62% <0%> (+1.3%) ⬆️
src/db.js 82.08% <0%> (+1.73%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a490abf...a346620. Read the comment docs.

@mrstegeman
Copy link
Contributor Author

@hobinjk This should be good now. The failures are because the intent-parser change is not yet merged.

@mrstegeman
Copy link
Contributor Author

Now we just have the failing UI test from #1338

Copy link
Contributor

@hobinjk hobinjk left a comment

Choose a reason for hiding this comment

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

LGTM with conflicts resolved and passing tests

@mrstegeman mrstegeman force-pushed the intent-parser branch 2 times, most recently from 6e2a892 to 139f14a Compare September 6, 2018 15:48
@mrstegeman mrstegeman merged commit 324a1df into WebThingsIO:master Sep 6, 2018
@ghost ghost removed the review label Sep 6, 2018
@mrstegeman mrstegeman deleted the intent-parser branch September 6, 2018 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants