Thanks 👍

It could be nice to add https://github.com/hadolint/hadolint as part of our automatic testing, e.g. in Travis or Docker builds. What do you think @Frenzie ?
P.S.: https://github.com/hadolint/hadolint/blob/master/docs/INTEGRATION.md

@Alkarex

It could be nice to add https://github.com/hadolint/hadolint as part of our automatic testing, e.g. in Travis or Docker builds.

Sure, I wasn't aware of it. Seems generally useful.

However, that curl=1.1.0 will break the script long before a change in the output from apt install would. :-)

I also noticed it :-) But it was just an example they used, I guess

@SuperSandro2000 or @Frenzie , if one of you is motivated enough to add haodlint to our CI, that would be welcome :-)

@SuperSandro2000 @Alkarex Maybe later today or tomorrow, not right now.

@SuperSandro2000 wrote #2451 (comment)

Btw I saw you wanted to add hadolint.
I would suggest this config https://github.com/SuperSandro2000/docker-images/blob/master/.hadolint.yaml of me and this install script of me https://github.com/SuperSandro2000/docker-images/blob/master/.travis.yml#L58 https://github.com/SuperSandro2000/docker-images/blob/master/.travis.yml#L61 .

That looks good at a first glance 👍 Would you mind sending a PR with that?

Yeah, that looks basically perfect except we'll want something closer to git ls-files --exclude='*Dockerfile*' --ignored or you'll exclude Docker/Dockerfile-Alpine and Docker/Dockerfile-QEMU-ARM.

Done in #2456

Cross-reference #2438