Style docker files consistently #107
ivanperez-keera
started this conversation in
Ideas
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I just noticed that docker files are not indented very consistently. For example, here the
&&
start a column zero:docker/space_robots/Dockerfile
Lines 81 to 83 in 76cd412
but just a little bit later, they start at column 2:
docker/space_robots/Dockerfile
Line 87 in 76cd412
I've always found that keeping a consistent style helps catch errors and propagate updates.
The following extra vertical space is not needed (and double space is not used elsewhere):
docker/space_robots/Dockerfile
Line 65 in 76cd412
The following invocations of
apt-get
passes the argument-y
last:docker/space_robots/Dockerfile
Line 67 in 76cd412
docker/space_robots/Dockerfile
Line 70 in 76cd412
Others pass it first:
docker/space_robots/Dockerfile
Line 47 in 76cd412
IMO the latter is preferable because it conveys the important information sooner and is less likely to be removed by mistake.
Beta Was this translation helpful? Give feedback.
All reactions