Skip to content

specify the shell for RUN commands#23

Merged
glennj merged 1 commit intoexercism:mainfrom
glennj:dockerfile
Mar 5, 2026
Merged

specify the shell for RUN commands#23
glennj merged 1 commit intoexercism:mainfrom
glennj:dockerfile

Conversation

@glennj
Copy link
Contributor

@glennj glennj commented Mar 5, 2026

This change doesn't affect the functionality of the docker build.

PR #22 was failing checks because it was seeing a "dockerfile parse error". This is even though the Dockerfile heredoc contained perfectly valid /bin/sh syntax.

Dockerfile:10
--------------------
   8 |     ENV DEBIAN_FRONTEND=noninteractive
   9 |     RUN << END_APT
  10 | >>>     set -e
  11 |         apt-get update -y
  12 |         apt-get install -y --no-install-recommends ca-certificates clang jq gawk locales curl
--------------------
ERROR: failed to build: failed to solve: dockerfile parse error on line 10: unknown instruction: set (did you mean user?)
Error: Process completed with exit code 1.

This change was suggested by Copilot, let's see if it helps.

@glennj glennj requested a review from a team as a code owner March 5, 2026 15:15
@glennj glennj merged commit 850e460 into exercism:main Mar 5, 2026
2 of 3 checks passed
@glennj glennj added x:rep/small Small amount of reputation x:type/ci Work on Continuous Integration (e.g. GitHub Actions workflows) labels Mar 5, 2026
@glennj glennj deleted the dockerfile branch March 5, 2026 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:rep/small Small amount of reputation x:type/ci Work on Continuous Integration (e.g. GitHub Actions workflows)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant