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

Update demo docstrings #268

Merged
merged 1 commit into from Sep 26, 2018

Conversation

Projects
None yet
3 participants
@Seth-Rothschild
Contributor

Seth-Rothschild commented Sep 26, 2018

Add contextual information to demo.load_retail and demo.load_flight that lists modifications we made to the CSVs. Related to #233, #201, #148 and #175
image

@codecov-io

This comment has been minimized.

codecov-io commented Sep 26, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #268   +/-   ##
=======================================
  Coverage   94.47%   94.47%           
=======================================
  Files          71       71           
  Lines        7690     7690           
=======================================
  Hits         7265     7265           
  Misses        425      425
Impacted Files Coverage Δ
featuretools/demo/flight.py 94.93% <ø> (ø) ⬆️
featuretools/demo/retail.py 93.75% <ø> (ø) ⬆️

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 ea0aee2...1e32b09. Read the comment docs.

@kmax12

This comment has been minimized.

Member

kmax12 commented Sep 26, 2018

Looks good, merging

@kmax12 kmax12 merged commit a0aed83 into master Sep 26, 2018

2 checks passed

ci/circleci Your tests passed on CircleCI!
Details
license/cla Contributor License Agreement is signed.
Details

@Seth-Rothschild Seth-Rothschild deleted the update_demo_docstring branch Sep 27, 2018

@kmax12 kmax12 referenced this pull request Sep 28, 2018

Merged

v0.3.1 #270

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