-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Codegen changes to support cwltool --fast-parse #156
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AttributeError: 'LoadingOptions' object has no attribute 'original_doc'
This pull request introduces 1 alert when merging c889cf2 into 5a00d2c - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging 346cbd5 into 5a00d2c - view on LGTM.com new alerts:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the outputSource
refscope fix broke the expression refactoring tool
cwl_utils.errors.WorkflowException: param out1 not found in class: Workflow
This pull request introduces 1 alert when merging 84c51ee into 5a00d2c - view on LGTM.com new alerts:
|
Codecov Report
@@ Coverage Diff @@
## main #156 +/- ##
==========================================
+ Coverage 29.53% 30.13% +0.59%
==========================================
Files 28 28
Lines 20763 21541 +778
Branches 5775 5981 +206
==========================================
+ Hits 6133 6491 +358
- Misses 13301 13627 +326
- Partials 1329 1423 +94
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
This pull request introduces 1 alert when merging a154797 into 5a00d2c - view on LGTM.com new alerts:
|
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <[email protected]>
This pull request introduces 1 alert when merging 4d1b211 into 53a129b - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging d028dff into 53a129b - view on LGTM.com new alerts:
|
Made via common-workflow-language/schema_salad#587
PR for CWL v1.1 schema refinementswe will use a custom branch: https://github.com/common-workflow-language/common-workflow-language/tree/codegenPR for CWL v1.0 schema refinementshttps://github.com/common-workflow-language/cwl-v1.1/tree/codegen