Skip to content

Commit 5acd8e6

Browse files
committed
[SPARK-56027][INFRA] Fix NoEmptyContinuation warning of spark-test-image/lint/Dockerfile
### What changes were proposed in this pull request? This PR removes a trailing backslash (`\`) on an empty continuation line in `dev/spark-test-image/lint/Dockerfile` to fix the `NoEmptyContinuation` warning from Dockerfile linters. This is the only instance of this pattern. - https://docs.docker.com/reference/build-checks/no-empty-continuation/ ### Why are the changes needed? The trailing `\` at line 69 is followed by an empty line, which triggers the `NoEmptyContinuation` warning. This is a no-op continuation that should be removed for clean lint results. ``` $ cd dev/spark-test-image/lint $ docker build . ... => WARN: NoEmptyContinuation: Empty continuation line (line 72) 0.0s ... ``` ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Verified by searching all Dockerfiles in the repository for the backslash-followed-by-empty-line pattern and confirming this was the only occurrence. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Code (claude-opus-4-6) Closes #54856 from dongjoon-hyun/SPARK-56027. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
1 parent b64383c commit 5acd8e6

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

dev/spark-test-image/lint/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ RUN apt-get update && apt-get install -y \
6666
RUN Rscript -e "install.packages(c('devtools', 'knitr', 'markdown', 'rmarkdown', 'testthat', 'remotes'), repos='https://cloud.r-project.org/')" \
6767
&& Rscript -e "devtools::install_version('pkgdown', version='2.0.1', repos='https://cloud.r-project.org')" \
6868
&& Rscript -e "devtools::install_version('preferably', version='0.4', repos='https://cloud.r-project.org')" \
69-
&& Rscript -e "devtools::install_version('lintr', version='2.0.1', repos='https://cloud.r-project.org')" \
69+
&& Rscript -e "devtools::install_version('lintr', version='2.0.1', repos='https://cloud.r-project.org')"
7070

7171
# See more in SPARK-39735
7272
ENV R_LIBS_SITE="/usr/local/lib/R/site-library:${R_LIBS_SITE}:/usr/lib/R/library"

0 commit comments

Comments
 (0)