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

Add specific docs blurb for -r #3566

Closed
nicoddemus opened this Issue Jun 11, 2018 · 6 comments

Comments

Projects
None yet
3 participants
@nicoddemus
Member

nicoddemus commented Jun 11, 2018

Follow up of #3546 (comment)

@pytestbot

This comment has been minimized.

Show comment
Hide comment
@pytestbot

pytestbot Jun 11, 2018

GitMate.io thinks possibly related issues are #1683 (Add "advanced fixtures" topic to docs), #83 (Add a list of included funcargs to the docs), #3141 (Add docs and examples for the '--pdbcls' option), #2670 (Add high-level docs page "how pytest works"?), and #795 (Docs typo).

pytestbot commented Jun 11, 2018

GitMate.io thinks possibly related issues are #1683 (Add "advanced fixtures" topic to docs), #83 (Add a list of included funcargs to the docs), #3141 (Add docs and examples for the '--pdbcls' option), #2670 (Add high-level docs page "how pytest works"?), and #795 (Docs typo).

@dhirensr

This comment has been minimized.

Show comment
Hide comment
@dhirensr

dhirensr Aug 29, 2018

Member

hey ,i am new to open source and would like to work on this ticket! could you please provide some reference what needs to be done exactly?

Member

dhirensr commented Aug 29, 2018

hey ,i am new to open source and would like to work on this ticket! could you please provide some reference what needs to be done exactly?

@nicoddemus

This comment has been minimized.

Show comment
Hide comment
@nicoddemus

nicoddemus Aug 30, 2018

Member

Thanks @dhirensr for stepping in!

I think we should add a dedicated section to the -r flag in usage.rst, possibly emphasizing that -ra is probably what users want in most cases.

Member

nicoddemus commented Aug 30, 2018

Thanks @dhirensr for stepping in!

I think we should add a dedicated section to the -r flag in usage.rst, possibly emphasizing that -ra is probably what users want in most cases.

@dhirensr

This comment has been minimized.

Show comment
Hide comment
@dhirensr

dhirensr Aug 31, 2018

Member

@nicoddemus : sent a PR for this! #3917
Please review and let me know if anything else needs to be added! 😄

Member

dhirensr commented Aug 31, 2018

@nicoddemus : sent a PR for this! #3917
Please review and let me know if anything else needs to be added! 😄

@dhirensr

This comment has been minimized.

Show comment
Hide comment
@dhirensr

dhirensr Sep 3, 2018

Member

@nicoddemus : i think we need to close this for now!

Member

dhirensr commented Sep 3, 2018

@nicoddemus : i think we need to close this for now!

@nicoddemus

This comment has been minimized.

Show comment
Hide comment
@nicoddemus

nicoddemus Sep 3, 2018

Member

Yep!

Btw, in the future, if you add "Fix #3566" to the description of the PR then the issue gets closed automatically when we merge the PR. 😉

See https://help.github.com/articles/closing-issues-using-keywords/

Thanks again!

Member

nicoddemus commented Sep 3, 2018

Yep!

Btw, in the future, if you add "Fix #3566" to the description of the PR then the issue gets closed automatically when we merge the PR. 😉

See https://help.github.com/articles/closing-issues-using-keywords/

Thanks again!

@nicoddemus nicoddemus closed this Sep 3, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment