Skip to content

bump docker/cli to v1.25 beta.1#11246

Closed
g0t4 wants to merge 1 commit intodocker:mainfrom
g0t4:bump-docker-cli-v1.25-beta1
Closed

bump docker/cli to v1.25 beta.1#11246
g0t4 wants to merge 1 commit intodocker:mainfrom
g0t4:bump-docker-cli-v1.25-beta1

Conversation

@g0t4
Copy link
Contributor

@g0t4 g0t4 commented Dec 4, 2023

What I did

Updated docker/cli to get access to RunAttach. Also updated docker/docker and added dependencies + code changes to get build working.

I could use some guidance as I am not familiar with these dependencies! Thanks

Related issue

This PR is needed for docker compose attach PR: #11181

Signed-off-by: Wes Higbee <wes.mcclure@gmail.com>
@g0t4
Copy link
Contributor Author

g0t4 commented Dec 4, 2023

@ndeloof once this is merged I will get the attach PR rebased.

@ndeloof
Copy link
Contributor

ndeloof commented Dec 5, 2023

this will unlock #11208 and #10939

@ndeloof
Copy link
Contributor

ndeloof commented Dec 5, 2023

just run go mod tidy and amend your commit, then we will be all good

@ndeloof ndeloof requested review from a team, glours, milas and ndeloof and removed request for a team December 5, 2023 08:47
@codecov
Copy link

codecov bot commented Dec 5, 2023

Codecov Report

Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (ce1ddb6) 56.81% compared to head (0853283) 56.80%.

Files Patch % Lines
pkg/compose/create.go 0.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11246      +/-   ##
==========================================
- Coverage   56.81%   56.80%   -0.02%     
==========================================
  Files         133      133              
  Lines       11512    11513       +1     
==========================================
- Hits         6541     6540       -1     
- Misses       4332     4334       +2     
  Partials      639      639              

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

@ndeloof
Copy link
Contributor

ndeloof commented Dec 5, 2023

I created #11247 to address linter issues

@milas
Copy link
Contributor

milas commented Dec 5, 2023

Merged with 56e2ad9 / #11247.

Thanks for the PR ❤️

@milas milas closed this Dec 5, 2023
@milas milas added the dependencies Pull requests that update a dependency file label Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants