-
Notifications
You must be signed in to change notification settings - Fork 14
More deploy stuff #1226
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
base: main
Are you sure you want to change the base?
More deploy stuff #1226
Conversation
|
Things I might look at here:
|
|
Think I've stumbled into a sizeable problem here. When loading the deploy project from the fs, there's no history. So you're always try to merge a zero-history workflow into one with a history. Which results in an error, because we can't guarantee no overwrite. But when deplying, don't we merge the fs into the local target? So why doesn't the local-merged Workflow have the history from the local file? Maybe this s just a tiny bug in workflow merging Is there also a problem here:
What we may need to do is: on fetch, if the local version is different to the new version coming down, we generate a new cli version in local history. Part of the problem there is that the version hashes aren't actually compatible. Hmm. Getting a bit tricky in here... If we track the version the local edit was forked from, like the source version, then we can work it out. But when do we record that? |
Likely to be a bitty PR with numerous small changes:
Closes #1230
--format stateon fetch to download a v1 state file. Not well tested. Useful for debugging only. Should write to .projects if-oisn't set, but that's not tested either. Proabably I should undocument this flagAI Usage
Please disclose how you've used AI in this work (it's cool, we just want to know!):
You can read more details in our Responsible AI Policy