Move sys.exit #1772
Move sys.exit #1772
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1772 +/- ##
==========================================
+ Coverage 82.47% 82.49% +0.01%
==========================================
Files 47 47
Lines 8195 8197 +2
Branches 2235 2235
==========================================
+ Hits 6759 6762 +3
Misses 941 941
+ Partials 495 494 -1
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 bd2dced into 60f0dac - view on LGTM.com new alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
|
This pull request introduces 1 alert when merging dcf5bb2 into 60f0dac - view on LGTM.com new alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog. |
|
Thank you @jfennick ; I see that you are the main author of https://github.com/jfennick/workflow_inference_compiler ; will you be submitting a talk to the 2023 CWL Conference about that tool? I'd love to see it! https://cwl.discourse.group/t/deadline-for-talk-proposals-is-monday-january-16th-2023/720 |
|
Glad you like it! FYI development is now moved to https://github.com/PolusAI/workflow-inference-compiler Pending paperworks and approvals from my management (likely), then yeah I'll be happy to give a talk. |
This PR simply moves the sys.exit() call so that users call call run() without exiting. I'm currently copy & pasting the body of run (without the sys.exit call of course), but that isn't ideal.
FYI the reason I'm interested in calling run() is because the normal way of using the cwltool Python API does not (easily) allow using all of the CLI arguments. (In particular, --cachedir and --provenance)