Skip to content
This repository has been archived by the owner on Oct 2, 2020. It is now read-only.

simplify hello.cwl #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

smoe
Copy link

@smoe smoe commented Feb 16, 2016

The first encounter with the cwl for executing something needs to be as simple as ... donno ... something really simple. The traditional "Hello world!" for instance does not need any input. It is a mere generator.

When executing the cwltool, I was asked if I wanted to to run #echocmd or #main. So, if #main is obsolete, which is good, then I strongly suggest not to have it with hello. With a bit of an imagination the CommandLineTool is a special kind of workflow, i.e. one that cannot nest or interconnect other workflows but wraps a tool. Then it is just fine to execute it direcly.

The input line should not be required for a command line. A missing such should just be interpreted as 'no inputs'.

I also removed the "stdout" line to make it more interactive, i.e. just like the original.

The first encounter with the cwl for executing something needs to be as simple as ... donno ... something really simple.  The traditional "Hello world!" for instance does not need any input. It is a mere generator.

When executing the cwltool, I was asked if I wanted to to run #echocmd or #main. So, if #main is obsolete, which is good, then I strongly suggest not to have it with hello. With a bit of an imagination the CommandLineTool is a special kind of workflow, i.e. one that cannot nest or interconnect other workflows but wraps a tool. Then it is just fine to execute it direcly.

The input line should not be required for a command line. A missing such should just be interpreted as 'no inputs'.

I also removed the "stdout" line to make it more interactive, i.e. just like the original.
smoe added a commit to smoe/cwltool that referenced this pull request Feb 16, 2016
A consequence of observations on the simplified hello.cwl and common-workflow-library/legacy#60 .
@mr-c
Copy link
Member

mr-c commented Apr 11, 2016

Hello @smoe , thank you for this PR.

I agree that one's first introduction should be simple. Have you seen @tetron 's new user guide? http://www.commonwl.org/draft-3/UserGuide.html

We could probably delete the 'hello' workflow given this new resource. What do you think?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants