Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
As discussed in #205, two CLI new commands to retrieve data for a whole Flow.
jf flow files getto fetch the files for all the jobs of a Flowjf flow dumpto dump the Flow internal document to a File. Optionally the file can contain also the Job data and create an additional file with all the outputs.@JaGeo can you please have a look at the code or test it to check if this is consistent with what you expected?
I think that in general this code can be used as a basis to create a personal CLI command, maybe tuning to specific needs (e.g. grouping all the code under a single command, deciding whether to interrupt the procedure in case of error, select a subset of files to download depending on the jobs, ...)