-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
Remove delete_existing
option; Introduce archive
#409
Conversation
Great work! Added some comments. Mainly I think we shouldn't let archive be a "step" but we just call it. Also, feel free to delete the "run_id" functionality. |
@@ -280,15 +295,17 @@ class Config(str, Enum): | |||
# Different configs of what steps to run | |||
STEPS = { | |||
Config.DEFAULT: [ | |||
archive, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's not make archive a step in this sense, but just call it before we loop over steps!
|
Merging! |
Thanks for the submission @azrv ! Do you want to join discord and DM me there? Happy to chat about future improvements! |
Already cleaned up run_id (was called run_prefix) |
Remove
delete_existing
run option, instead archive memory and workspace on every run;archive
is a step since some flows don't generate code and therefore should not archive whateverRelates to #388