search

common-workflow-language / cwltool

Common Workflow Language reference implementation
https://cwltool.readthedocs.io/
Apache License 2.0
333 stars 230 forks source link

patch dockerFile build #1979

Closed misterbrandonwalker closed 4 months ago

misterbrandonwalker commented 7 months ago

Don't let dockerFile tag be dependent on force_pull or pull_image booleans

codecov[bot] commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.72%. Comparing base (81b5f84) to head (f43df0f).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1979 +/- ## ========================================== - Coverage 83.85% 83.72% -0.14% ========================================== Files 46 46 Lines 8245 8245 Branches 2190 2190 ========================================== - Hits 6914 6903 -11 - Misses 855 862 +7 - Partials 476 480 +4 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

mr-c commented 4 months ago

Thank you @misterbrandonwalker !

misterbrandonwalker commented 4 months ago

[heart] Brandon Walker reacted to your message:

From: Michael R. Crusoe @.> Sent: Wednesday, May 8, 2024 7:35:57 AM To: common-workflow-language/cwltool @.> Cc: Brandon Walker @.>; Mention @.> Subject: Re: [common-workflow-language/cwltool] patch dockerFile build (PR #1979)

Thank you @misterbrandonwalkerhttps://github.com/misterbrandonwalker !

— Reply to this email directly, view it on GitHubhttps://github.com/common-workflow-language/cwltool/pull/1979#issuecomment-2099937452, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKNB26N4DCKONW4CAQ2WQTTZBHIV3AVCNFSM6AAAAABD25AQUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJZHEZTONBVGI. You are receiving this because you were mentioned.Message ID: @.***>