Skip to content

useDockerDefaultOrServicePlatform fct should return service.platform if defined#10017

Merged
ndeloof merged 1 commit intodocker:v2from
glours:use-service-platform-to-build-when-up
Nov 25, 2022
Merged

useDockerDefaultOrServicePlatform fct should return service.platform if defined#10017
ndeloof merged 1 commit intodocker:v2from
glours:use-service-platform-to-build-when-up

Conversation

@glours
Copy link
Copy Markdown
Contributor

@glours glours commented Nov 23, 2022

and present in the build.platforms list (or if the list is empty)

Signed-off-by: Guillaume Lours [email protected]

What I did
Move the check of the service.platform presence inside the service.build.platforms list with the test of the list size, to be sure we always return the service.plaform value when present in the build platforms list or when this list is empty

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

@glours glours requested review from a team, rumpl and vvoland November 23, 2022 21:48
@glours glours self-assigned this Nov 23, 2022
@glours glours requested review from StefanScherer, laurazard, milas, ndeloof, nicksieger and ulyssessouza and removed request for a team November 23, 2022 21:48
Copy link
Copy Markdown
Contributor

@vvoland vvoland left a comment

Choose a reason for hiding this comment

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

LGTM

…if defined

and present in the build.platforms list (or if the list is empty)

Signed-off-by: Guillaume Lours <[email protected]>
@glours glours force-pushed the use-service-platform-to-build-when-up branch from a27f45f to 0ee9afa Compare November 24, 2022 15:44
@ndeloof ndeloof merged commit 10a5d99 into docker:v2 Nov 25, 2022
@glours glours deleted the use-service-platform-to-build-when-up branch January 11, 2023 14:59
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