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

Drop Python 3.6 support #35214

Merged
merged 68 commits into from
Aug 11, 2020
Merged

Drop Python 3.6 support #35214

merged 68 commits into from
Aug 11, 2020

Conversation

fangchenli
Copy link
Member

@fangchenli fangchenli commented Jul 10, 2020

  • closes DEPS: drop 3.6 #34472
  • passes black pandas
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • whatsnew entry

@jreback jreback added this to the 1.2 milestone Jul 10, 2020
@jreback jreback added the Dependencies Required and optional dependencies label Jul 10, 2020
Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

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

thanks, we would look to do this post 1.1 release, e.g. target 1.2 (so soonish)

setup.py Show resolved Hide resolved
@fangchenli

This comment has been minimized.

@jorisvandenbossche jorisvandenbossche changed the title Drop 3.6 Drop Python 3.6 support Jul 11, 2020
@fangchenli fangchenli deleted the drop-3.6 branch August 11, 2020 01:39
@@ -1151,6 +1152,7 @@ def test_chunks_have_consistent_numerical_type(all_parsers):
assert result.a.dtype == float


@pytest.mark.skip("unreliable test #35214")
Copy link
Member

Choose a reason for hiding this comment

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

  1. should this be xfail?
  2. i think the solution will end up involving using td.check_file_leaks in a lot of places. Is there a non-verbose way to run it once at the module level?

Copy link
Member Author

Choose a reason for hiding this comment

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

  1. Yes, it should be xfail. I'll submit a PR to fix this.
  2. Haven't thought about this. Might need some discussion.

simonjayhawkins added a commit to simonjayhawkins/pandas that referenced this pull request Aug 14, 2020
@ivanovmg ivanovmg mentioned this pull request Sep 27, 2020
simonjayhawkins added a commit to simonjayhawkins/pandas that referenced this pull request Nov 26, 2020
shlomiku added a commit to alpacahq/alpaca-trade-api-python that referenced this pull request Dec 8, 2020
@lucasjamar lucasjamar mentioned this pull request Jul 5, 2021
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dependencies Required and optional dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DEPS: drop 3.6
9 participants