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 support for EOL Python 3.4 #226

Merged
merged 1 commit into from May 6, 2020
Merged

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Dec 3, 2019

Also rename appveyor.yml to .appveyor.yml to hide it with the other dotfiles, and update some links. --> split out to #233.

@codecov-io
Copy link

codecov-io commented Dec 3, 2019

Codecov Report

Merging #226 into master will decrease coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #226      +/-   ##
==========================================
- Coverage   82.06%   81.96%   -0.11%     
==========================================
  Files          55       55              
  Lines       10161    10161              
  Branches     1141     1141              
==========================================
- Hits         8339     8328      -11     
- Misses       1558     1569      +11     
  Partials      264      264

@jayvdb
Copy link
Contributor

jayvdb commented Dec 21, 2019

Note that there is an unreleased fix for py34, which really should be released before dropping support for py34.

@hugovk
Copy link
Member Author

hugovk commented Dec 21, 2019

Sounds reasonable. Is there a schedule for the next release?

@hugovk
Copy link
Member Author

hugovk commented Jan 19, 2020

I see py 1.8.1 has now been released with the Python 3.4 fix:

py/CHANGELOG

Lines 1 to 5 in 1058b29

1.8.1 (2019-12-27)
==================
- Handle ``FileNotFoundError`` when trying to import pathlib in ``path.common``
on Python 3.4 (#207).

Ready for merge?

@blueyed
Copy link
Contributor

blueyed commented Jan 19, 2020

@hugovk there are some good changes in here (e.g. f07e105), but IMHO there is no need to drop support if it is not really necessary.
AFAICS py is in maintenance mode, so there is nothing like e.g. type annotations being added to the code etc, which would benefit from it. (btw: typing stubs are being worked on in #232).
So I suggest to get your non-dropping changes merged via new PR maybe by themselves already.
Just my 2 cents however.

@hugovk
Copy link
Member Author

hugovk commented Jan 19, 2020

@blueyed Yes, the main benefit is dropping a few slots from the CI. But if the code isn't updated much, that's no as important.

However, 3.4 is now failing on AppVeyor so it looks like some dependencies need pinning to maintain support.

So I suggest to get your non-dropping changes merged via new PR maybe by themselves already.

Good idea, please see PR #233.

@blueyed
Copy link
Contributor

blueyed commented Jan 19, 2020

@hugovk

Yes, the main benefit is dropping a few slots from the CI.

That could still be done, of course (or reduced). But removing it from install_requires for example is not really necessary (currently).

@blueyed
Copy link
Contributor

blueyed commented Jan 19, 2020

For reference: py34 on AppVeyor fails via pytest installing colorama:

Collecting colorama; sys_platform == "win32" (from pytest~=2.9.0)
  Downloading https://files.pythonhosted.org/packages/31/cb/88c908c1be067fb6bacd3d7488ccab1a212533767b951933aac3d22648e2/colorama-0.4.2-py2.py3-none-any.whl
colorama requires Python '>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, !=3.4.*' but the running Python is 3.4.4
You are using pip version 10.0.1, however version 19.1.1 is available.
You should consider upgrading via the 'python -m pip install --upgrade pip' command.

blueyed added a commit to blueyed/py that referenced this pull request Jan 19, 2020
It is failing there, and we do not want to put effort into supporting
it.

Ref: pytest-dev#226 (comment)
@blueyed
Copy link
Contributor

blueyed commented Jan 19, 2020

I've created #235 to remove py34 from AV.

blueyed added a commit that referenced this pull request Jan 19, 2020
It is failing there, and we do not want to put effort into supporting
it.

Ref: #226 (comment)
@hugovk
Copy link
Member Author

hugovk commented Jan 19, 2020

Rebased to the remove the non-drop changes in #233.

Copy link
Member

@asottile asottile left a comment

Choose a reason for hiding this comment

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

otherwise looks good

setup.py Outdated Show resolved Hide resolved
@blueyed
Copy link
Contributor

blueyed commented Jan 31, 2020

Please squash.

I'd like to be able doing it here, but cannot...

@hugovk
Copy link
Member Author

hugovk commented Jan 31, 2020

Done!

@nicoddemus nicoddemus merged commit f7b20d2 into pytest-dev:master May 6, 2020
@hugovk hugovk deleted the rm-3.4 branch May 6, 2020 12:33
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.

None yet

6 participants