Skip to content

compose-file: add some options that were missing in changes - #10680

Merged
usha-mandya merged 1 commit into
docker:masterfrom
thaJeztah:compose_build_network
Apr 22, 2020
Merged

compose-file: add some options that were missing in changes#10680
usha-mandya merged 1 commit into
docker:masterfrom
thaJeztah:compose_build_network

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

I noticed the network option for build was not documented, which was added in compose-file 2.2 (docker/compose#4997) and 3.4 (docker/cli#430), then saw some missing entries for the v3.1 and v3.2 format, so added those (probably more changes may be needed, but it didn't go through all versions)

@usha-mandya@ulyssessouza PTAL

@thaJeztahthaJeztah added the area/compose Relates to docker-compose.yml spec or docker-compose binary label Apr 21, 2020
@thaJeztah
thaJeztahforce-pushed the compose_build_network branch from dfb7338 to 0e43307CompareApril 21, 2020 18:56
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
thaJeztahforce-pushed the compose_build_network branch from 0e43307 to 3caa2caCompareApril 21, 2020 18:57
@netlify

netlifyBot commented Apr 21, 2020

Copy link
Copy Markdown

Deploy preview for docsdocker ready!

Built with commit dfb7338

https://deploy-preview-10680--docsdocker.netlify.app

@netlify

netlifyBot commented Apr 21, 2020

Copy link
Copy Markdown

Deploy preview for docsdocker ready!

Built with commit 3caa2ca

https://deploy-preview-10680--docsdocker.netlify.app

@usha-mandyausha-mandya left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@ulyssessouzaulyssessouza left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@usha-mandya
usha-mandya merged commit 72f90f5 into docker:masterApr 22, 2020
@thaJeztah
thaJeztah deleted the compose_build_network branch April 22, 2020 13:20
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/composeRelates to docker-compose.yml spec or docker-compose binary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@thaJeztah@ulyssessouza@usha-mandya