Skip to content

Update cwltool version to the latest#2469

Merged
mr-c merged 1 commit intomasterfrom
cwltool-update
Nov 26, 2018
Merged

Update cwltool version to the latest#2469
mr-c merged 1 commit intomasterfrom
cwltool-update

Conversation

@psafont
Copy link
Copy Markdown
Contributor

@psafont psafont commented Nov 16, 2018

Right the referenced cwltool version has not been yet released

@ghost ghost assigned psafont Nov 16, 2018
@ghost ghost added the in progress label Nov 16, 2018
@psafont psafont requested a review from mr-c November 16, 2018 12:59
@DailyDreaming
Copy link
Copy Markdown
Member

DailyDreaming commented Nov 17, 2018

@psafont Pip error is: No matching distribution found for cwltool==1.0.20181115172452

The latest pip shows is 1.0.20181116032456.

It's also necessary to change the test commit hash when updating cwl (the git hash associated with the commit for that release):
https://github.com/DataBiosphere/toil/blob/master/src/toil/test/cwl/cwlTest.py#L50

This ensures the latest cwl tests are run with the corresponding version.

Thanks for this, it's a much appreciated update!

Update the has for having the latest conformance tests as well
@DailyDreaming
Copy link
Copy Markdown
Member

DailyDreaming commented Nov 20, 2018

@psafont This looks good to me, as long as @mr-c approves I'll merge.

@mr-c mr-c merged commit 886c305 into master Nov 26, 2018
@ghost ghost removed the in progress label Nov 26, 2018
@mr-c mr-c deleted the cwltool-update branch November 26, 2018 09:57
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