Skip to content

Remove implementation of __getslice__#1958

Merged
nicoddemus merged 1 commit intopytest-dev:masterfrom
rowillia:master
Sep 24, 2016
Merged

Remove implementation of __getslice__#1958
nicoddemus merged 1 commit intopytest-dev:masterfrom
rowillia:master

Conversation

@rowillia
Copy link
Copy Markdown
Contributor

__getslice__ has been Deprecated since Python 2.0 and is removed in Python 3. See
https://docs.python.org/2/reference/datamodel.html#object.__getslice__

Unfortunately, Python 2 will still dispatch to __getslice__ over __getitem__, See
http://bugs.python.org/issue2041, which causes Warnings when running with -3 in 2.7.

`__getslice__` has been Deprecated since Python 2.0 and is removed in Python 3.  See
https://docs.python.org/2/reference/datamodel.html#object.__getslice__

Unfortunately, Python 2 will still dispatch to `__getslice__` over `__getitem__`, See
http://bugs.python.org/issue2041, which causes Warnings when running with `-3` in 2.7.
@nicoddemus
Copy link
Copy Markdown
Member

Merging because the failures are not related to the patch, will investigate why the failures are happening later.

@nicoddemus nicoddemus merged commit 09bcf7f into pytest-dev:master Sep 24, 2016
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at ?% when pulling f1c4cfe on rowillia:master into e5deb8a on pytest-dev:master.

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.

4 participants