ci: Drop default for `inputs.command` in `run-in-docker-action`
What changed, and why it matters
This is a minor GitHub Actions CI cleanup. It removes a default shell command from a reusable workflow action and instead makes callers explicitly provide the command. There is no security-relevant change to the actual Bitcoin cryptography code or to how containers are built or executed.
No action required. This is a benign CI refactoring.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies .github/actions/run-in-docker-action/action.yml to change the command input from optional (defaulting to ./ci/ci.sh) to required with no default. The CI workflow .github/workflows/ci.yml is updated to pass command: ./ci/ci.sh explicitly. The behavior is identical; only the coupling between the action and its caller is reduced. No commands, permissions, secrets, container images, or execution logic are altered.
Changed components
.github/actions/run-in-docker-action/action.yml.github/workflows/ci.ymlInspect captured patch +2 / −2
diff --git a/.github/actions/run-in-docker-action/action.yml b/.github/actions/run-in-docker-action/action.yml
index d3ab44b..5d46ca1 100644
--- a/.github/actions/run-in-docker-action/action.yml
+++ b/.github/actions/run-in-docker-action/action.yml
@@ -10,8 +10,7 @@ inputs:
default: ${{ runner.arch }}
command:
description: 'A command to run in a container'
- required: false
- default: ./ci/ci.sh
+ required: true
runs:
using: "composite"
steps:
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 8327294..8508224 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -117,6 +117,7 @@ jobs:
uses: ./.github/actions/run-in-docker-action
with:
dockerfile: ./ci/linux-debian.Dockerfile
+ command: ./ci/ci.sh
- &PRINT_LOGS
name: Print logs
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.