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

Merge master into features #5921

Closed
wants to merge 30 commits into from
Closed

Merge master into features #5921

wants to merge 30 commits into from

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Oct 6, 2019

Ref: #5902

Could be done after a release also, feel free to do so then instead, @nicoddemus.

jamescooke and others added 30 commits September 20, 2019 18:38
Fix logging doc: change x.level to x.levelno
Recently sometimes Azure has failed with:

++ curl -s https://codecov.io/bash
bash: /dev/fd/63: No such file or directory

This attempts to fix this by modifying report-coverage.sh slightly.
Use sys.executable to detect which python we should actually be testing.
Change report-coverage.sh in attempt to fix Azure
Make sure to quote `sys.executable` as we're running a shell
This should result in retries of 1+2+4+8+16+32 = 63s.

Ref: pytest-dev#5869 (comment)
…items-5884

Fix --setup-only and --setup-show for custom pytest items
This allows us to use python3.7+ to use tox
…pytester

Fix attribute docs in _pytest.pytester
Update doc: pytest section in setup.cfg
attrs 19.2 deprecated cmp in favor of the dataclass-ish eq/order duo.

This causes deprecation warnings that in turn break some of the cool new deep
object comparisons. Since we at attrs expected this to be a problem, it shipped
with helpers to write backward and forward compatible code.

This PR uses that and avoids changed to minimal versions.
Fix warnings with attrs 19.2 and fix object assertions
@blueyed blueyed changed the title Mm Merge master into features Oct 6, 2019
@nicoddemus
Copy link
Member

nicoddemus commented Oct 6, 2019

Yeah I was waiting the release before doing it, let me open a new one, thanks. 👍

#5923

@nicoddemus nicoddemus closed this Oct 6, 2019
@blueyed blueyed deleted the mm branch October 6, 2019 21:00
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.

8 participants