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

Split requirements to separate files and pin to latest versions #15

Merged
merged 3 commits into from
Oct 12, 2017

Conversation

joshblum
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented Oct 12, 2017

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #15   +/-   ##
=======================================
  Coverage   86.44%   86.44%           
=======================================
  Files          73       73           
  Lines        6549     6549           
=======================================
  Hits         5661     5661           
  Misses        888      888

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 984510e...6e05966. Read the comment docs.

@joshblum joshblum requested a review from kmax12 October 12, 2017 01:21
README.md Outdated
@@ -3,6 +3,7 @@
[![Circle CI](https://circleci.com/gh/Featuretools/featuretools.svg?maxAge=2592000&style=shield)](https://circleci.com/gh/Featuretools/featuretools)
[![Coverage Status](https://codecov.io/gh/Featuretools/featuretools/branch/master/graph/badge.svg)](https://codecov.io/gh/Featuretools/featuretools)
[![PyPI version](https://badge.fury.io/py/featuretools.svg?maxAge=2592000)](https://badge.fury.io/py/featuretools)
[![Requirements Status](https://requires.io/github/Featuretools/featuretools/requirements.svg?branch=master)](https://requires.io/github/Featuretools/featuretools/requirements/?branch=master)
Copy link
Contributor

Choose a reason for hiding this comment

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

let's remove the badge for now

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@kmax12
Copy link
Contributor

kmax12 commented Oct 12, 2017

looks good to me

@kmax12 kmax12 merged commit 3aaab45 into alteryx:master Oct 12, 2017
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