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

setup.py: freeze smart_open dependency to git version #16

Merged
merged 1 commit into from Dec 1, 2015

Conversation

Projects
None yet
3 participants
@kernc
Copy link
Member

commented Dec 1, 2015

Revert once smart_open>=1.3.1 releases.

Fixes #15

setup.py: freeze smart_open dependency to git version
Revert once smart_open>=1.3.1 releases.

Fixes #15
@codecov-io

This comment has been minimized.

Copy link

commented Dec 1, 2015

Current coverage is 56.65%

Merging #16 into master will not affect coverage as of dd6e829

@@            master     #16   diff @@
======================================
  Files            8       8       
  Stmts          383     383       
  Branches        87      87       
  Methods          0       0       
======================================
  Hit            217     217       
  Partial         23      23       
  Missed         143     143       

Review entire Coverage Diff as of dd6e829


Uncovered Suggestions

  1. +3.66% via ...econtrib/text/lda.py#1...14
  2. +2.61% via ...econtrib/text/lda.py#37...46
  3. +2.35% via ...ontrib/text/stats.py#44...52
  4. See 7 more...

Powered by Codecov. Updated on successful CI builds.

nikicc added a commit that referenced this pull request Dec 1, 2015

Merge pull request #16 from kernc/freeze-smart_open
setup.py: freeze smart_open dependency to git version

@nikicc nikicc merged commit d14edf0 into biolab:master Dec 1, 2015

3 checks passed

codecov/patch coverage not affected
Details
codecov/project 56.65% remains the same compared to 2a7a90e
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@kernc kernc referenced this pull request Dec 11, 2015

Closed

Orange-Text #859

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.