Skip to content

identify services to build and don't display 'building' if none#11209

Merged
ndeloof merged 1 commit intodocker:mainfrom
glours:hide-building-if-no-service-to-build
Nov 21, 2023
Merged

identify services to build and don't display 'building' if none#11209
ndeloof merged 1 commit intodocker:mainfrom
glours:hide-building-if-no-service-to-build

Conversation

@glours
Copy link
Copy Markdown
Contributor

@glours glours commented Nov 20, 2023

What I did
First identify services to be build and early exit if none

Related issue

fixes #11174

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

@glours glours self-assigned this Nov 20, 2023
@glours glours requested review from milas and ndeloof November 20, 2023 17:21
@glours glours force-pushed the hide-building-if-no-service-to-build branch from 48171ce to 15b2299 Compare November 20, 2023 17:25
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 20, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a1b7bee) 56.66% compared to head (15b2299) 56.80%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11209      +/-   ##
==========================================
+ Coverage   56.66%   56.80%   +0.13%     
==========================================
  Files         133      133              
  Lines       11451    11474      +23     
==========================================
+ Hits         6489     6518      +29     
+ Misses       4326     4321       -5     
+ Partials      636      635       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ndeloof ndeloof merged commit f557220 into docker:main Nov 21, 2023
@glours glours deleted the hide-building-if-no-service-to-build branch June 30, 2025 21:21
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.

[BUG] compose run display [+] Building X.Ys (n/m) on stdout even if there is nothing to build

2 participants