Skip to content

Commit c09185c

Browse files
committed
Remove explicit install of eslint inside of tidy's Dockerfile
`tidy` will already install it (when needed) due to it being in `package.json`
1 parent 00426d6 commit c09185c

File tree

2 files changed

+0
-6
lines changed

2 files changed

+0
-6
lines changed

src/ci/docker/host-x86_64/tidy/Dockerfile

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,6 @@ COPY scripts/nodejs.sh /scripts/
2828
RUN sh /scripts/nodejs.sh /node
2929
ENV PATH="/node/bin:${PATH}"
3030

31-
# Install eslint
32-
COPY host-x86_64/tidy/eslint.version /tmp/
33-
3431
COPY scripts/sccache.sh /scripts/
3532
RUN sh /scripts/sccache.sh
3633

@@ -40,8 +37,6 @@ RUN pip3 install --no-deps --no-cache-dir --require-hashes -r /tmp/reuse-require
4037

4138
COPY host-x86_64/pr-check-1/validate-toolstate.sh /scripts/
4239

43-
RUN bash -c 'npm install -g eslint@$(cat /tmp/eslint.version)'
44-
4540
# NOTE: intentionally uses python2 for x.py so we can test it still works.
4641
# validate-toolstate only runs in our CI, so it's ok for it to only support python3.
4742
ENV SCRIPT TIDY_PRINT_DIFF=1 python2.7 ../x.py test \

src/ci/docker/host-x86_64/tidy/eslint.version

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)