Skip to content

networks: fix ipam config example#417

Merged
ndeloof merged 1 commit intocompose-spec:masterfrom
milas:fix-ipam-example
Sep 7, 2023
Merged

networks: fix ipam config example#417
ndeloof merged 1 commit intocompose-spec:masterfrom
milas:fix-ipam-example

Conversation

@milas
Copy link
Copy Markdown
Member

@milas milas commented Sep 6, 2023

This was missing one level of indentation.

Signed-off-by: Milas Bowman <[email protected]>
@milas milas added the bug Something isn't working label Sep 6, 2023
@milas milas self-assigned this Sep 6, 2023
milas added a commit to milas/docker.github.io that referenced this pull request Sep 6, 2023
See https://github.com/compose-spec/compose-spec/blob/796804a634e4a1fd48a70f0b0b0e02f1a3a97890/06-networks.md#L168-L196

(Note: there's a typo there too, which is being fixed as
compose-spec/compose-spec#417. The indentation is wrong, but the
description of the fields/hierarchy is correct.)

Signed-off-by: Milas Bowman <[email protected]>
Copy link
Copy Markdown
Member

@laurazard laurazard left a comment

Choose a reason for hiding this comment

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

Nice <3

Copy link
Copy Markdown
Contributor

@aevesdocker aevesdocker left a comment

Choose a reason for hiding this comment

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

Thanks you Milas!

@ndeloof ndeloof merged commit 6042e09 into compose-spec:master Sep 7, 2023
aevesdocker pushed a commit to docker/docs that referenced this pull request Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants