-
-
Notifications
You must be signed in to change notification settings - Fork 20
Use yaml.dump over old dump command, stripped double .cwl, and fixed import issues #287
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
Use yaml.dump over old dump command, stripped double .cwl, and fixed import issues #287
Conversation
7584f8a
to
c903624
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #287 +/- ##
==========================================
+ Coverage 34.33% 34.53% +0.20%
==========================================
Files 30 30
Lines 35362 35375 +13
Branches 9553 9555 +2
==========================================
+ Hits 12142 12218 +76
+ Misses 20327 20269 -58
+ Partials 2893 2888 -5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…imports Added cwlformat to requirements (needed for --pretty parameter), Co-authored-by: Michael R. Crusoe <[email protected]>
a2aa6eb
to
4736b18
Compare
4736b18
to
c9ac6c3
Compare
c9ac6c3
to
5508c43
Compare
@alexiswl Thanks again, |
Thanks @mr-c I think we should run a try-exception to better inform the user to install with cwl-utils[pretty] if they do try to run cwl-graph-split with the pretty parameter. |
Sure, can you send a PR? Then I'll review, merge, and make a new release |
Latest version of cwl-utils has common-workflow-language/cwl-utils#288 and common-workflow-language/cwl-utils#287
Added cwlformat to requirements (needed for --pretty parameter),
Resolves #285
Resolves #286