Skip to content
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.

Sign up for GitHub

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

Jump to bottom

add additional code checks to automated testing #286

Open
3 tasks
mr-c opened this issue Feb 8, 2017 · 0 comments
Open
3 tasks

add additional code checks to automated testing #286

mr-c opened this issue Feb 8, 2017 · 0 comments

Comments

@mr-c
Copy link
Member

mr-c commented Feb 8, 2017

inspired by #285

  • unused imports
  • pep8
  • empty lists in function defs
@mr-c mr-c changed the title Additional code checks add additional code checks to automated testing Mar 3, 2017
qiukunlong pushed a commit to qiukunlong/cwltool that referenced this issue Mar 25, 2017
Squashed 'cwltool/schemas/' changes from 46ce0c3..2cf0896
79ef5c3 
2cf0896 Merge pull request common-workflow-language#289 from denis-yuen/patch-1
68f5f90 Merge pull request common-workflow-language#291 from bmeg/master
a82cf0d Merge https://github.com/bmeg/common-workflow-language
e7325dc Adding docker hint for conformance test
70501d5 Merge pull request common-workflow-language#290 from alaindomissy/patch-1
58a9354 Update concepts.md
1b052b5 Fix a typo
4a02bc7 Test that expressionLib requirement of individual tool step overrides expressionLib of workflow.
80e2501 add path to record-job3.yaml
c9df19c Add checksum to cwl.output.json results.
e604399 Add tests that $HOME and $TMPDIR are set correctly according to spec.
a479bbd Add checksums and sizes to secondaryFiles in draft-3 tests.
dfcbfeb Merge pull request common-workflow-language#286 from common-workflow-language/test-nested-array
91bb7b4 Test case for command line generation of array-of-arrays.
5baf924 Merge pull request common-workflow-language#285 from common-workflow-language/checksum-secondary
1c02570 Merge pull request common-workflow-language#282 from common-workflow-language/test-inputbinding-dir
1835540 Add checksums to secondaryFiles and files in Directory objects.
04d9dac Add test for command line inputBinding of Directory input parameter.
8053c4b Merge pull request common-workflow-language#279 from simonovic86/patch-1
168d1a2 Update README.md
e727a46 Merge pull request common-workflow-language#277 from common-workflow-language/add-biostars-link
bfddadc fix typo
9ec9642 add biostars link
b67a4cf Merge pull request common-workflow-language#272 from StarvingMarvin/master
570ae5f Merge pull request common-workflow-language#266 from common-workflow-language/remove-empty-baseCommand
721ffdc Merge remote-tracking branch 'origin/master' into remove-empty-baseCommand
6931eeb minor userguide fixes
a296c5f Remove "baseCommand: []" from tests.

git-subtree-dir: cwltool/schemas
git-subtree-split: 2cf0896
tetron added a commit that referenced this issue Jul 23, 2017
@tetron
Merge pull request #286 from common-workflow-language/test-nested-array
dfcbfeb 
Test case for command line generation of array-of-arrays.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mr-c