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

rename tmpdir.rst to tmp_path.rst #8897

Closed
graingert opened this issue Jul 12, 2021 · 2 comments · Fixed by #8905
Closed

rename tmpdir.rst to tmp_path.rst #8897

graingert opened this issue Jul 12, 2021 · 2 comments · Fixed by #8905
Labels
type: docs documentation improvement, missing or needing clarification

Comments

@graingert
Copy link
Member

graingert commented Jul 12, 2021

but also with a permanent 308 redirect from tmpdir to tmp_path so the link juice is conserved

currently the tmp_path docs are hosted at: https://docs.pytest.org/en/6.2.x/tmpdir.html#the-tmp-path-fixture

but really it would be better to use https://docs.pytest.org/en/6.2.x/tmp_path.html#the-tmp-path-fixture

@nicoddemus nicoddemus added the type: docs documentation improvement, missing or needing clarification label Jul 12, 2021
@graingert
Copy link
Member Author

I'd make a PR for this, but I don't know how to configure the redirect

@nicoddemus
Copy link
Member

The redirect is configured in the RTD interface, we can do that after the PR gets merged. 👍

graingert added a commit to graingert/pytest that referenced this issue Jul 13, 2021
graingert added a commit to graingert/pytest that referenced this issue Jul 13, 2021
The-Compiler added a commit to The-Compiler/pytest that referenced this issue Nov 8, 2021
The-Compiler added a commit that referenced this issue Nov 10, 2021
* Remove changelog entry for #8251

Reverted in #8903

* Move #9202 changelog to to trivial

This won't concern users of pytest

* Streamline deprecation changelogs/docs

* Remove #8994 changelog

This is an impovement for a warning introduced in this release, so including it in a changelog against the last release seems confusing.

* Remove #9241 changelog

This is an impovement for a doc update introduced in this release, so including it in a changelog against the last release seems confusing. The issue number also seems about something different.

* Remove #8897 changelog

Empty file...

* Various minor changelog fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: docs documentation improvement, missing or needing clarification
Projects
Development

Successfully merging a pull request may close this issue.

2 participants