Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #103 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 1 1
Lines 2 2
=========================================
Hits 2 2 ☔ View full report in Codecov by Sentry. |
action.yml
Outdated
| default: 'yes' | ||
| threads: | ||
| description: 'Value passed to the --threads flag. Default value: auto.' | ||
| default: 'auto' |
There was a problem hiding this comment.
auto isn't the julia default unfortunately, so this will need to be a bit cleverer. Basically not set the arg by default
There was a problem hiding this comment.
Ah, yeah.
This might suffice:
default: "${JULIA_NUM_THREADS:=1}"
Or I can do a
if [[ -n ${{ input.threads }} ]]; then
julia ... --threads ${{ input.threads }} -e ...
else
julia ...
fi
There was a problem hiding this comment.
Better to populate a var for the threads arg and interpolate it in, as if any other args need the same behavior different if paths for the top level cmd would make for a mess of if statements
There was a problem hiding this comment.
Sorry, I meant a different approach like threads_arg = inputs.threads == "default" ? "" : "--threads=${{ inputs.threads}}
Then interpolate threads_arg into the cmd, making use of it being empty to avoid setting the arg
|
Looks like
|
90693c8 to
6d8e68b
Compare
No description provided.