fix: use JSON notation for HEALTHCHECK CMD (DL3025) #15

Merged
jcabillot merged 1 commits from fix/hadolint-DL3025-healthcheck-json into main 2026-08-02 13:04:41 -04:00
Owner

Fix hadolint warning DL3025: Use arguments JSON notation for CMD and ENTRYPOINT arguments.

Before:

CMD curl -f http://localhost:8787/health || exit 1

After:

CMD ["curl", "-f", "http://localhost:8787/health"]

curl -f already returns a non-zero exit code on failure, so the || exit 1 is redundant.

Fix hadolint warning DL3025: Use arguments JSON notation for CMD and ENTRYPOINT arguments. Before: ``` CMD curl -f http://localhost:8787/health || exit 1 ``` After: ``` CMD ["curl", "-f", "http://localhost:8787/health"] ``` `curl -f` already returns a non-zero exit code on failure, so the `|| exit 1` is redundant.
opencodecabilloteu added 1 commit 2026-08-02 12:57:27 -04:00
fix: use JSON notation for HEALTHCHECK CMD (DL3025)
PR Checks / build (pull_request) Successful in 5m38s
PR Checks / hadolint (pull_request) Successful in 31s
aa7b1f8eb3
opencodecabilloteu requested review from jcabillot 2026-08-02 12:57:53 -04:00
jcabillot merged commit 9452efcc18 into main 2026-08-02 13:04:41 -04:00
jcabillot deleted branch fix/hadolint-DL3025-healthcheck-json 2026-08-02 13:04:43 -04:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: web/hermes-agent-webui#15