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.0-3.2 support code from setup.py #1678

Merged
merged 4 commits into from
Jun 30, 2016

Conversation

RonnyPfannschmidt
Copy link
Member

addresses #1627

@coveralls
Copy link

coveralls commented Jun 27, 2016

Coverage Status

Coverage remained the same at 92.09% when pulling 0e20c14 on RonnyPfannschmidt:drop-python30-32 into ccd395f on pytest-dev:features.

@@ -27,10 +27,12 @@

Thanks to `@RedBeardCode`_ for the PR(`#1664`_)

* removed support code for python 3 < 3.3 addresssing (`#1627`_)
Copy link
Member

Choose a reason for hiding this comment

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

s/sss/ss/ (hah!)

Copy link
Member

Choose a reason for hiding this comment

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

😅

@nicoddemus
Copy link
Member

LGTM, feel free to merge after fixing the small typo. 😁

extras_require[':sys_platform=="win32"'] = ['colorama']
else:
if sys.version_info < (2, 7) or (3,) <= sys.version_info < (3, 2):
if sys.version_info < (2, 7):
Copy link
Member

Choose a reason for hiding this comment

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

This might be as well sys.version_info[:2] == (2, 6), for consistency with the marker usage in 54

@The-Compiler
Copy link
Member

There is more stuff which should be changed/dropped to properly remove support

@RonnyPfannschmidt
Copy link
Member Author

@The-Compiler in this pr i just want to kill the documented/normal install one

i plan to add some more PR's to remove strange code

for example i stumbled on a python 2.4 support thing ^^

@The-Compiler
Copy link
Member

  • pytest.org still says "runs on Posix/Windows, Python 2.6-3.5"
  • _pytest/assertion/rewrite.py has various 3.0/3.1/3.2 support code (search for REWRITE_NEWLINES and 3.2+)
  • README.rst says 3.2+
  • testing/code/test_source.py has an xfail for < (3,2) which can be removed

@The-Compiler
Copy link
Member

Okay, but at least adjust the documentation then.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.09% when pulling e877e25 on RonnyPfannschmidt:drop-python30-32 into ccd395f on pytest-dev:features.

@RonnyPfannschmidt
Copy link
Member Author

im on it

@coveralls
Copy link

coveralls commented Jun 27, 2016

Coverage Status

Coverage remained the same at 92.09% when pulling 2a43237 on RonnyPfannschmidt:drop-python30-32 into ccd395f on pytest-dev:features.

@coveralls
Copy link

coveralls commented Jun 27, 2016

Coverage Status

Coverage remained the same at 92.09% when pulling 2a43237 on RonnyPfannschmidt:drop-python30-32 into ccd395f on pytest-dev:features.

@nicoddemus
Copy link
Member

Oh sorry, I didn't have the code with me to check the docs... agree that we should properly drop the support in a single PR (setup.py, docs, code, etc)

@RonnyPfannschmidt
Copy link
Member Author

should i drop anything else? (atm i really only want to drop docs and mentions, then phase bits of support code out in the 3.x series)

@The-Compiler
Copy link
Member

I'm with @nicoddemus - not sure what the rationale for only dropping half of it is? If we drop the code in this PR as well, we can be sure it's actually gone and we don't forget about it 😉

@RonnyPfannschmidt
Copy link
Member Author

ok

@RonnyPfannschmidt
Copy link
Member Author

after investigating the support code for getstatementrange, it addresses python 2.4 and should be removed in a different pr

as for the assertion rewriter, - the exact code are conditionals also valid for python 2.6, and i would like to leave them until python 2.6 removal

@The-Compiler
Copy link
Member

as for the assertion rewriter, - the exact code are conditionals also valid for python 2.6, and i would like to leave them until python 2.6 removal

Oh, right, didn't think of that.

Could you still adjust the xfail in test_source though?

@pytest.mark.xfail("sys.version_info[:3] < (2,7,0) or "
    "((3,0) <= sys.version_info[:2] < (3,2))")
def test_source_of_class_at_eof_without_newline(tmpdir):

@RonnyPfannschmidt
Copy link
Member Author

i think i would like to remove that as we remove python2.6 as well, since we right now still have a testsuite that can run on 3.2, does any enterprise distro keep users on that?

@The-Compiler
Copy link
Member

Huh - what's the point of having a testsuite passing on 3.2 when we say we drop 3.2 support? 😉

As for conservative distros, I'm pretty sure they'll either stick with Python 2 altogether, or not going to use pytest 3.0. Hell, pip dropped it 😉

@RonnyPfannschmidt
Copy link
Member Author

true, will change the condition :)

@coveralls
Copy link

coveralls commented Jun 30, 2016

Coverage Status

Coverage remained the same at 92.09% when pulling 1451a1a on RonnyPfannschmidt:drop-python30-32 into ccd395f on pytest-dev:features.

@The-Compiler
Copy link
Member

The-Compiler commented Jun 30, 2016

I'd like to see appveyor pass here, but it doesn't, see #1690.

@nicoddemus to the rescue? Or anyone else on Windows? 😆

edit: heh, back to normal right now 😆

@nicoddemus
Copy link
Member

😬

@The-Compiler The-Compiler merged commit 7927dff into pytest-dev:features Jun 30, 2016
@The-Compiler
Copy link
Member

Thanks! ✨

@RonnyPfannschmidt RonnyPfannschmidt deleted the drop-python30-32 branch August 21, 2016 18:07
LeSuisse added a commit to LeSuisse/confiture that referenced this pull request Oct 25, 2017
This contribution drop the execution of the unit tests on
Python 3.2. Python 3.2 is not supported by pytest anymore [1]
and there are no more security updates [2]. confiture should
not encourage the use of an abandoned version of Python.

Also, this contribution adds the current Python 3 version (3.6)
to the list of the tested environments.

[1] pytest-dev/pytest#1678
[2] https://www.python.org/dev/peps/pep-0392/#lifespan
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

4 participants