Skip to content

[PR #12706/72c682ff backport][8.3.x] Fix for Verbose mode not having default value on getoption call works on issue #9422#12778

Merged
nicoddemus merged 1 commit into8.3.xfrom
patchback/backports/8.3.x/72c682ff9773ad2690711105a100423ebf7c7c15/pr-12706
Sep 5, 2024
Merged

[PR #12706/72c682ff backport][8.3.x] Fix for Verbose mode not having default value on getoption call works on issue #9422#12778
nicoddemus merged 1 commit into8.3.xfrom
patchback/backports/8.3.x/72c682ff9773ad2690711105a100423ebf7c7c15/pr-12706

Conversation

@patchback
Copy link

@patchback patchback bot commented Sep 5, 2024

This is a backport of PR #12706 as merged into main (72c682f).

Followed the suggestions on this issue and added a default value to the verbose mode calls as it does in some of the other sections of the code.

Refs #9422

@GTowers1 @patchback
Streamline checks for verbose option (#12706)
71507a2 
Instead of calling `Config.option.verbose`, call the new `Config.get_verbosity` function to determine the verbosity level.

This enables pytest to run correctly with the terminal plugin disabled.

Fix #9422

(cherry picked from commit 72c682f)
@patchback patchback bot mentioned this pull request Sep 5, 2024
Merged
@nicoddemus nicoddemus enabled auto-merge (squash) September 5, 2024 11:04
@nicoddemus nicoddemus merged commit bc913d1 into 8.3.x Sep 5, 2024
@nicoddemus nicoddemus deleted the patchback/backports/8.3.x/72c682ff9773ad2690711105a100423ebf7c7c15/pr-12706 branch September 5, 2024 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nicoddemus nicoddemus nicoddemus approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nicoddemus @GTowers1