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

doc: add the explanation on how to read/write strategy fields #1746

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zenixls2
Copy link
Collaborator

DOC: add documentation on how to print/edit fields from a strategy in interactive scenarios (ex: telegram or slack)

@bbgokarma-bot
Copy link

Welcome back! @zenixls2, This pull request may get 261 BBG.

Copy link

codecov bot commented Sep 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 24.11%. Comparing base (1f8b2b3) to head (271eafa).
Report is 5 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1746   +/-   ##
=======================================
  Coverage   24.11%   24.11%           
=======================================
  Files         615      615           
  Lines       37541    37544    +3     
=======================================
+ Hits         9052     9055    +3     
  Misses      27691    27691           
  Partials      798      798           

see 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 80430fe...271eafa. Read the comment docs.

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