Skip to content

#6289: Add new example to XFAIL documentation #6685

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

Merged
merged 8 commits into from
Feb 19, 2020

Conversation

nathanielcompton
Copy link
Contributor

Based on the discussions linked in open issue #6289 , I added an additional example to the xfail documentation.

@nathanielcompton
Copy link
Contributor Author

@graingert or @nicoddemus would one of you please help me understand the failing codecov test? I've tried to grok the details/output on the codecov page. Unless I'm mistaken, I believe I've only edited files that aren't even tracked by codecov. Is this correct?

@bluetech
Copy link
Member

@nathanielcompton Don't worry about this check, it's a little touch-and-go. Since you don't change any code, only docs, you should ignore it in this PR.

@nicoddemus
Copy link
Member

Great review @bluetech.

LGTM with @bluetech's suggestions applied.

@nathanielcompton
Copy link
Contributor Author

@nicoddemus and @bluetech : Ready for re-review when you get time. Thanks for the good feedback.

Copy link
Member

@bluetech bluetech left a comment

Choose a reason for hiding this comment

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

Thanks @nathanielcompton, it looks good to me now, except two minor nits.

nathanielcompton and others added 2 commits February 18, 2020 18:42
@nicoddemus nicoddemus merged commit 7c09d88 into pytest-dev:master Feb 19, 2020
@nicoddemus
Copy link
Member

Thanks @bluetech and @nathanielcompton!

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.

3 participants