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

fix for Python 3.7: re._pattern_type no longer exist #849

Merged
merged 4 commits into from
Nov 3, 2019

Conversation

smola
Copy link
Contributor

@smola smola commented Oct 31, 2019

What do these changes do?

  • re._pattern_type was removed in Python 3.7. re.Pattern can be used
    instead.
  • Try re.Pattern first, and fall back to re._pattern_type otherwise.

Related issue number

  • passes flake8 modin
  • passes black --check modin (except for unrelated file)
  • [ ] tests added and passing: Not applicable, tests are present, but they need to run with Python 3.7.

* re._pattern_type was removed in Python 3.7. re.Pattern can be used
instead.
* Try re.Pattern first, and fall back to re._pattern_type otherwise.
@codecov
Copy link

codecov bot commented Oct 31, 2019

Codecov Report

Merging #849 into master will decrease coverage by 0.05%.
The diff coverage is 83.33%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #849      +/-   ##
=========================================
- Coverage   90.85%   90.8%   -0.06%     
=========================================
  Files          47      47              
  Lines        5656    5656              
=========================================
- Hits         5139    5136       -3     
- Misses        517     520       +3
Impacted Files Coverage Δ
modin/pandas/series.py 93.34% <83.33%> (-0.14%) ⬇️
modin/engines/ray/pandas_on_ray/frame/partition.py 88.23% <0%> (-1.18%) ⬇️
modin/engines/ray/generic/io.py 90.98% <0%> (-0.22%) ⬇️

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 6a51d9a...fb095d1. Read the comment docs.

Copy link
Collaborator

@devin-petersohn devin-petersohn left a comment

Choose a reason for hiding this comment

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

Thanks @smola, I went ahead and made some other changes that resolve #855.

@devin-petersohn devin-petersohn merged commit a824069 into modin-project:master Nov 3, 2019
@smola smola deleted the py37-re branch November 4, 2019 08:44
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.

2 participants