aw-was-here commented on a change in pull request #201:
URL: https://github.com/apache/yetus/pull/201#discussion_r524886103



##########
File path: precommit/src/main/shell/test-patch-docker/Dockerfile
##########
@@ -324,40 +324,6 @@ RUN apt-get -q update && apt-get -q install 
--no-install-recommends -y \
         yamllint==1.24.2 \
     && rm -rf /root/.cache \
     && mv /usr/local/bin/pylint /usr/local/bin/pylint3

Review comment:
       Taking this out leaves just `/usr/local/bin/pylint` . I kept it like 
this so that `/usr/local/bin/pylint` is a symlink to `/usr/local/bin/pylint3` 
(which is the real binary) in case someone wants to easily override the version 
in the container or add python2 back or for python 4 or whatever.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to