CROM-4572 Allow describing wdl with zipped imports. #7133
Conversation
This file contains 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
|
NOTE: While I need to describe these WDLs using our Cromwell instance, there is a CROM-4572 ticket already created that was previously closed, and this comment mentions eliminating zip imports. While that's being figured out, I'm contributing this patch JIC someone else wants to cherry-pick this for their instance. |
kshakir
force-pushed
the
ks_server_describe_zip
branch
from
1d1c815 to
ca14a65
Compare
kshakir
changed the title
kshakir
force-pushed
the
ks_server_describe_zip
branch
from
ca14a65 to
a8c64c3
Compare
|
We're definitely looking to deprecate zip imports. They've never been supported in Terra, and aren't planned for Terra on Azure either. The current work in this area is support for workflows in private Github repos – so that HTTP imports with without having to make WDLs public. See #7392. |
kshakir
force-pushed
the
ks_server_describe_zip
branch
from
a8c64c3 to
5573b3f
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.