-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Futures Curve: minor cosmetic improvements #4901
Futures Curve: minor cosmetic improvements #4901
Conversation
…port. change column header to Last Price. set x axis of curve chart to correspond to contract months
Hi @skhan59bbk, thanks for the PR and the improvement! The code linting is failing because it has not been formatted according to Black's wishes. Run it from the command line at the root of the OpenBBTerminal folder:
However, because the expected outputs have been changed, you will also have to re-record the unit test cassettes so that the changes to the results are captured.
This will update a few files in the |
…han59bbk/OpenBBTerminal into hotfix/futures-contract-names
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## develop #4901 +/- ##
========================================
Coverage 58.28% 58.29%
========================================
Files 588 588
Lines 53667 53665 -2
========================================
+ Hits 31280 31282 +2
+ Misses 22387 22383 -4
☔ View full report in Codecov by Sentry. |
Catering for #4886