Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[PR #12679/dc756f41 backport][8.3.x] Remove quotes from example tmp_path_retention_policy#12683

Merged
nicoddemus merged 1 commit into
8.3.xfrom
patchback/backports/8.3.x/dc756f4117f7cbabbe0866bab1fe562ac287cc03/pr-12679
Aug 3, 2024
Merged

[PR #12679/dc756f41 backport][8.3.x] Remove quotes from example tmp_path_retention_policy#12683
nicoddemus merged 1 commit into
8.3.xfrom
patchback/backports/8.3.x/dc756f4117f7cbabbe0866bab1fe562ac287cc03/pr-12679

Conversation

@patchback

@patchback patchback Bot commented Aug 3, 2024

Copy link
Copy Markdown

This is a backport of PR #12679 as merged into main (dc756f4).

Previously the docs contained an example for tmp_path_retention_policy which included quotes around the value. If the configuration was actually set with this value. The following error would be encountered

INTERNALERROR> ValueError: tmp_path_retention_policy must be either all, failed, none. Current input: "all".

This is because the value of the field was not being unwrapped from the quotes when parsed.

This commit removes the quotes from the example so that if used in the configuration file, no error will occur.

closes #12678

…mple

Remove quotes from example tmp_path_retention_policy

(cherry picked from commit dc756f4)
@nicoddemus nicoddemus enabled auto-merge (squash) August 3, 2024 14:10
@nicoddemus nicoddemus merged commit 32f9951 into 8.3.x Aug 3, 2024
@nicoddemus nicoddemus deleted the patchback/backports/8.3.x/dc756f4117f7cbabbe0866bab1fe562ac287cc03/pr-12679 branch August 3, 2024 14:15
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.

1 participant