Add --include-deps to push command

[gferon]

What I did

Add --include-deps to the push sub-command, similar to what pull can do. I believe the default behavior is a little bit weird since you can't explicitly push the built image of a service that has declared dependencies. That being said, this change will break backwards compatibility, so I feel like we could

Cute animal tax - a photo of my containerized cat:

[ndeloof]

While I don't expect many users to rely on this, I'm a bit concerned about another backward compatibility break.
Maybe better introduce --no-deps to mimic same option used by up?
Also, to avoid code duplication, could define a func to disable all services but selected ones

[codecov]

Codecov Report

Base: 76.98% // Head: 75.79% // Decreases project coverage by -1.19% ⚠️

Coverage data is based on head (91f7dcb) compared to base (8c39b5b).
Patch has no changes to coverable lines.

Additional details and impacted files

@@            Coverage Diff             @@
##               v2   #10044      +/-   ##
==========================================
- Coverage   76.98%   75.79%   -1.20%     
==========================================
  Files           2        2              
  Lines         252      252              
==========================================
- Hits          194      191       -3     
- Misses         51       53       +2     
- Partials        7        8       +1     

Impacted Files	Coverage Δ
pkg/e2e/framework.go	74.04% <0.00%> (-1.28%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[gferon]

While I don't expect many users to rely on this, I'm a bit concerned about another backward compatibility break.

Fair point, I can definitely move it to a --no-deps flag instead, but this would be inconsistent with docker-compose pull. What about setting the value of this flag to true instead and let users pass --include-deps=false instead?

Also, to avoid code duplication, could define a func

Sure, can you point me to a good place to do this? I'm not familiar with this codebase to select a proper location myself.

[ndeloof]

I didn't noticed --include-deps flag existed for pull, makes more sense now. Both should be aligned. Minor compatibility break probably is acceptable from this point of view.

you can just extract https://github.com/docker/compose/blob/v2/cmd/compose/up.go#L57-L66 into a func inside up.go and call it from both command - by the way, can also use it for pull where code already is duplicated

[ndeloof]

also need to regenerate docs to reflect new flag using make docs

[gferon]

@ndeloof not sure why you meant up.go, I extracted the function in pull.go - if you really wanted it to be in up.go I can change it.

Otherwise, this should be ready.