[BugFix] Move showwarning
To Outside catch_warnings
Context Block
#7058
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why?:
obb.etf.info("AAPL,XLE", provider="yfinance")
function hangs when equity symbol and ETF symbol are combined... #7042CommandRunner
.What?:
showwarning
, and appending the list of warnings to theOBBject
, to outside of thecatch_warnings
context block.Impact:
show_warning
user preference to default asFalse
.Testing Done:
openbb-core
tests.