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

Docs updates #708

Merged
merged 2 commits into from
Feb 24, 2025
Merged

Docs updates #708

merged 2 commits into from
Feb 24, 2025

Conversation

davidlatte
Copy link
Collaborator

@davidlatte davidlatte commented Feb 22, 2025

Rebuilding published docs after Order Management update changed lots of things.

Description by Korbit AI

What change is being made?

Remove the is_backtesting entry from strategy properties documentation and update the backtesting function documentation to refine parameters and return types.

Why are these changes being made?

The removal of the is_backtesting entry is needed to correct the documentation, as it may no longer be relevant or accurate. Updates to the backtesting function documentation reflect changes for clarity in describing parameters and return types, addressing developer needs for comprehensible and accurate documentation.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

@davidlatte davidlatte requested a review from grzesir as a code owner February 22, 2025 01:19
Copy link
Contributor

korbit-ai bot commented Feb 22, 2025

Korbit doesn't automatically review large (3000+ lines changed) pull requests such as this one. If you want me to review anyway, use /korbit-review.

@grzesir grzesir merged commit f29c073 into dev Feb 24, 2025
1 check passed
@davidlatte davidlatte deleted the docsrc_update branch February 24, 2025 06:47
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