Skip to content

Address dependencies needed to allow Docker image to build#224

Open
jwalkerwelltok wants to merge 2 commits intodatabricks:mainfrom
jwalkerwelltok:master
Open

Address dependencies needed to allow Docker image to build#224
jwalkerwelltok wants to merge 2 commits intodatabricks:mainfrom
jwalkerwelltok:master

Conversation

@jwalkerwelltok
Copy link
Copy Markdown

@jwalkerwelltok jwalkerwelltok commented Apr 30, 2019

This is the smallest patch I could come up with to allow for this to build in Docker. This addresses issue #166. Both click and tabulate are imported in the application directly, so those libraries were added to dev-requirements.txt instead of tox-requirements.txt
@codecov-io
Copy link
Copy Markdown

codecov-io commented Apr 30, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #224   +/-   ##
=======================================
  Coverage   83.08%   83.08%           
=======================================
  Files          31       31           
  Lines        1969     1969           
=======================================
  Hits         1636     1636           
  Misses        333      333

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 cb22ab2...2416475. Read the comment docs.
@jwalkerwelltok
Copy link
Copy Markdown
Author

jwalkerwelltok commented May 1, 2019

@andrewmchen, do you have a chance to review this? I ask because you were involved in discussion on #167
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@jwalkerwelltok @codecov-io