Skip to content
This repository was archived by the owner on Jun 3, 2024. It is now read-only.

Fix for Dockerfile smell DL3015#347

Merged
mboersma merged 1 commit intodeis:mainfrom
grosa1:fix/dockerfile-smell-DL3015
Aug 15, 2023
Merged

Fix for Dockerfile smell DL3015#347
mboersma merged 1 commit intodeis:mainfrom
grosa1:fix/dockerfile-smell-DL3015

Conversation

@grosa1
Copy link
Copy Markdown
Contributor

@grosa1 grosa1 commented Apr 13, 2023

Hi!
The Dockerfile placed at "rootfs/Dockerfile" contains the best practice violation DL3015 detected by the hadolint tool.

The smell DL3015 occurs when the apt tool is used to install packages without the "--no-install-recommends" flag. This flag is recommended to be used to avoid installing additional packages not explicitly requested.
In this pull request, we propose a fix for that smell generated by our fixing tool. We have verified that the patch is correct before opening the pull request.
To fix this smell, specifically, the "--no-install-recommends" flag is added to the apt-get install command.

This change is only aimed at fixing that specific smell. If the fix is not valid or useful, please briefly indicate the reason and suggestions for possible improvements.

Thanks in advance

Signed-off-by: Giovanni Rosa <g.rosa1@studenti.unimol.it>
Copy link
Copy Markdown
Member

@mboersma mboersma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Thank you!

@mboersma mboersma merged commit fa6ddde into deis:main Aug 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants