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

[PR #12912/a1a49183 backport][8.3.x] Improve docs on basetemp and retention #12928

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Oct 31, 2024

This is a backport of PR #12912 as merged into main (a1a4918).

Improve coverage of current handling of --basetemp option and its lack of retention functionality.

  • [ ] Include documentation when adding new features.
  • [ ] Include new tests or update existing tests when applicable.
  • Allow maintainers to push and squash when merging my commits.
  • Add text like closes #XYZW to the PR description and/or commits (where XYZW is the issue number). See the github docs for more information.
  • Create a new changelog file in the changelog folder, with a name like <ISSUE NUMBER>.<TYPE>.rst. See changelog/README.rst for details.
  • Add yourself to AUTHORS in alphabetical order.

Addresses (part of) issue #10829:
the --basetemp option and the default retention feature can not be combined currently. This is completely unclear from the current docs. While I initially requested to allow the combination of retention with --basetemp in #10829, this PR just attempts better document the current behavior.

Improve coverage of current handling of `--basetemp` option and its lack of retention functionality.

Also document `PYTEST_DEBUG_TEMPROOT`.

Related to #10829

---------

Co-authored-by: Bruno Oliveira <[email protected]>
(cherry picked from commit a1a4918)
@patchback patchback bot mentioned this pull request Oct 31, 2024
4 tasks
@nicoddemus nicoddemus enabled auto-merge (squash) October 31, 2024 10:29
@nicoddemus nicoddemus merged commit 7aeb72b into 8.3.x Oct 31, 2024
29 checks passed
@nicoddemus nicoddemus deleted the patchback/backports/8.3.x/a1a491837b35b719d77b4f03be318b505d495386/pr-12912 branch October 31, 2024 10:46
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.

2 participants