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

introduce --dry-run mode #532

Closed
wants to merge 5 commits into from

Conversation

junyuanz1
Copy link
Collaborator

When I try to test specific changes with a live account, I have to comment on a few functions.

in this PR I am going to add a --dry-run CLI flag so we can see the result without executing them.

Dry run enabled, no trades will be executed.
                                                                                   Order Summary                                                                                    
         ╷          ╷                                                                                                                                     ╷        ╷         ╷      
  Symbol │ Exchange │ Contract                                                                                                                            │ Action │ Price   │ Qty  
╺━━━━━━━━┿━━━━━━━━━━┿━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┿━━━━━━━━┿━━━━━━━━━┿━━━━━╸
  SGOV   │ SMART    │ Stock(conId=424099317, symbol='SGOV', exchange='SMART', primaryExchange='ARCA', currency='USD', localSymbol='SGOV',                 │ BUY    │ $100.38 │ 144  
         │          │ tradingClass='SGOV')                                                                                                                │        │         │      
         ╵          ╵                                                                                                                                     ╵        ╵         ╵      
ThetaGang is done, shutting down! Cya next time. ✨

@junyuanz1 junyuanz1 marked this pull request as ready for review December 4, 2024 04:27
@junyuanz1 junyuanz1 closed this by deleting the head repository Dec 4, 2024
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.

1 participant