Skip to content

Update dockerfile syntax to latest stable, and rename docs Dockerfile#9711

Merged
milas merged 2 commits intodocker:v2from
thaJeztah:update_dockerfile_syntax
Aug 3, 2022
Merged

Update dockerfile syntax to latest stable, and rename docs Dockerfile#9711
milas merged 2 commits intodocker:v2from
thaJeztah:update_dockerfile_syntax

Conversation

@thaJeztah
Copy link
Member

What I did

update dockerfiles to use latest stable syntax

Some Dockerfiles were pinned to a minor release, which meant they wouldn't be updated to get the latest stable syntax (and fixes), and one Dockerfile used the "labs" variant to use the HEREDOC syntax, which has now been promoted to the stable syntax.

docs: rename Dockerfile

There's no other Dockerfiles in the same path, so the "docs" prefix was redundant.

Related issue

(not mandatory) A picture of a cute animal, if possible in relation with what you did

Some Dockerfiles were pinned to a minor release, which meant they
wouldn't be updated to get the latest stable syntax (and fixes),
and one Dockerfile used the "labs" variant to use the HEREDOC syntax,
which has now been promoted to the stable syntax.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
There's no other Dockerfiles in the same path, so the "docs"
prefix was redundant.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Copy link
Member

@nicksieger nicksieger left a comment

Choose a reason for hiding this comment

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

👍

@milas milas merged commit 92f32b5 into docker:v2 Aug 3, 2022
@thaJeztah thaJeztah deleted the update_dockerfile_syntax branch August 3, 2022 20:56
laurentsimon pushed a commit to laurentsimon/compose that referenced this pull request Aug 10, 2022
…ker#9711)

* update dockerfiles to use latest stable syntax

Some Dockerfiles were pinned to a minor release, which meant they
wouldn't be updated to get the latest stable syntax (and fixes),
and one Dockerfile used the "labs" variant to use the HEREDOC syntax,
which has now been promoted to the stable syntax.

* docs: rename Dockerfile

There's no other Dockerfiles in the same path, so the "docs"
prefix was redundant.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants