Skip to content

[28.x backport] add local fork of github.com/docker/docker/builder/remotecontext - #6265

Merged
austinvazquez merged 2 commits into
docker:28.xfrom
thaJeztah:28.x_backport_fork_remotecontext
Aug 15, 2025
Merged

[28.x backport] add local fork of github.com/docker/docker/builder/remotecontext#6265
austinvazquez merged 2 commits into
docker:28.xfrom
thaJeztah:28.x_backport_fork_remotecontext

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

backport:

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 26.53061% with 36 lines in your changes missing coverage. Please review.

Files with missing linesPatch %Lines
cli/command/image/build.go32.25%18 Missing and 3 partials ⚠️
cli/command/image/build/context_detect.go0.00%15 Missing ⚠️

📢 Thoughts on this report? Let us know!

thaJeztahand others added 2 commits August 14, 2025 21:50
Removes direct imports of github.com/docker/docker/builder in
the image package, to be moved later.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 260f1db)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
add local fork of github.com/docker/docker/builder/remotecontext
(cherry picked from commit 8c317ad)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah

thaJeztah commented Aug 14, 2025

Copy link
Copy Markdown
MemberAuthor

FWIW; as the original PR merged from another repository and I didn't want to cherry-pick all those commits, I just cherry-picked the merge-commit from master; 8c317ad

@thaJeztah
thaJeztah marked this pull request as ready for review August 14, 2025 20:02
@austinvazquez
austinvazquez merged commit 2dc9daa into docker:28.xAug 15, 2025
108 of 110 checks passed
@thaJeztah
thaJeztah deleted the 28.x_backport_fork_remotecontext branch August 15, 2025 13:08
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@thaJeztah@codecov-commenter@austinvazquez