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

Update pytest and clean up tests a bit #903

Merged

Conversation

devin-petersohn
Copy link
Collaborator

What do these changes do?

Related issue number

  • passes flake8 modin
  • passes black --check modin
  • tests added and passing

@devin-petersohn devin-petersohn added this to the 0.6.4 milestone Dec 12, 2019
@codecov
Copy link

codecov bot commented Dec 12, 2019

Codecov Report

Merging #903 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #903      +/-   ##
==========================================
+ Coverage   87.71%   87.71%   +<.01%     
==========================================
  Files          54       54              
  Lines        5737     5739       +2     
==========================================
+ Hits         5032     5034       +2     
  Misses        705      705
Impacted Files Coverage Δ
modin/pandas/dataframe.py 89.06% <100%> (+0.02%) ⬆️

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 2cde933...dc56727. Read the comment docs.

@devin-petersohn devin-petersohn merged commit 1c40056 into modin-project:master Dec 13, 2019
@devin-petersohn devin-petersohn mentioned this pull request Jan 6, 2020
9 tasks
@devin-petersohn devin-petersohn mentioned this pull request Jan 16, 2020
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.

1 participant