Skip to content

Add --print-supported-versions option that prints supported CWL versions #402

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

Merged
merged 3 commits into from
May 31, 2017
Merged

Add --print-supported-versions option that prints supported CWL versions #402

merged 3 commits into from
May 31, 2017

Conversation

tom-tan
Copy link
Member

@tom-tan tom-tan commented May 31, 2017

Currently the only way to check whether the certain version of CWL file is supported by cwltool is to run cwltool with CWL files and to check its output.

This request adds --print-supported-versions that prints supported CWL versions.
We can check the CWL versions supported by cwltool by using this option.

Here is the output of --print-supported-versions option:

$ ./cwltool.py --print-supported-versions
./cwltool.py 1.0.20170516234254
draft-2
draft-3
draft-3.dev1
draft-3.dev2
draft-3.dev3
draft-3.dev4
draft-3.dev5
draft-4.dev1
draft-4.dev2
draft-4.dev3
v1.0
v1.0.dev4
v1.1.0-dev1

@cwl-bot
Copy link

cwl-bot commented May 31, 2017

Can one of the admins verify this patch?

@mr-c
Copy link
Member

mr-c commented May 31, 2017

Jenkins, okay to test

@mr-c
Copy link
Member

mr-c commented May 31, 2017

Thank you @tom-tan for this PR!
FYI, many of those versions require the --enable-dev flag -- perhaps it would be best to only print the full list if --enable-dev was also passed?

@tom-tan
Copy link
Member Author

tom-tan commented May 31, 2017

Thank you for your advice.

I will update this PR to take it into account.

@tom-tan
Copy link
Member Author

tom-tan commented May 31, 2017

Done!

I update supportedCWLversions to take enable_dev flag.

  • If enable_dev is true, it returns all supported CWL versions (include devel versions).
  • Otherwise it returns all draft and released versions without devel versions.

@mr-c mr-c merged commit 81a6e9e into common-workflow-language:master May 31, 2017
@mr-c
Copy link
Member

mr-c commented May 31, 2017

Great, thanks again!

@tom-tan tom-tan deleted the print-cwl-versions branch May 31, 2017 08:39
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.

3 participants