Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Only install Python 3 version of adapt parser. #790

Merged
merged 1 commit into from
Mar 22, 2018

Conversation

mrstegeman
Copy link
Contributor

The intent parser runs with Python 3, so there's no reason to have
a Python2 version of adapt parser.

The intent parser runs with Python 3, so there's no reason to have
a Python2 version of adapt parser.
@ghost ghost assigned mrstegeman Mar 22, 2018
@ghost ghost added the review label Mar 22, 2018
@codecov-io
Copy link

codecov-io commented Mar 22, 2018

Codecov Report

Merging #790 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #790   +/-   ##
=======================================
  Coverage   78.81%   78.81%           
=======================================
  Files          85       85           
  Lines        3465     3465           
  Branches      455      455           
=======================================
  Hits         2731     2731           
  Misses        674      674           
  Partials       60       60

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 1c42f06...326a0a6. Read the comment docs.

@mrstegeman mrstegeman changed the title Only install Python 3 version of adapte parser. Only install Python 3 version of adapt parser. Mar 22, 2018
@mrstegeman mrstegeman merged commit cfe1f8d into WebThingsIO:master Mar 22, 2018
@ghost ghost removed the review label Mar 22, 2018
@mrstegeman mrstegeman deleted the adapt-parser branch March 22, 2018 14:55
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants