Skip to content

replace dockerfile/dockerignore with patternmatcher/ignorefile - #4514

Merged
thaJeztah merged 2 commits into
docker:masterfrom
thaJeztah:patternmatcher_dockerignore
Aug 23, 2023
Merged

replace dockerfile/dockerignore with patternmatcher/ignorefile#4514
thaJeztah merged 2 commits into
docker:masterfrom
thaJeztah:patternmatcher_dockerignore

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

vendor: github.com/moby/patternmatcher v0.6.0

  • integrate frontend/dockerfile/dockerignore from buildkit

full diff: moby/patternmatcher@v0.5.0...v0.6.0

replace dockerfile/dockerignore for patternmatcher/ignorefile

The BuildKit dockerignore package was migrated to the patternmatcher
repository / module. This patch updates our uses of the BuildKit package
with its new location.

A small local change was made to keep the format of the existing error message,
because the "ignorefile" package is slightly more agnostic in that respect
and doesn't include ".dockerignore" in the error message.

- Description for the changelog

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

@thaJeztahthaJeztah added this to the 25.0.0 milestone Aug 22, 2023
@thaJeztahthaJeztah changed the title replace dockerfile/dockerignore for patternmatcher/ignorefilereplace dockerfile/dockerignore with patternmatcher/ignorefileAug 22, 2023
@thaJeztah
thaJeztahforce-pushed the patternmatcher_dockerignore branch 2 times, most recently from cd98295 to c085b31CompareAugust 22, 2023 22:27
@codecov-commenter

codecov-commenter commented Aug 22, 2023

Copy link
Copy Markdown

Codecov Report

Merging #4514 (5bff123) into master (b3f8207) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Additional details and impacted files
@@ Coverage Diff @@## master #4514 +/- ##
==========================================
- Coverage 59.35% 59.34% -0.01% 
==========================================
Files 288 288 Lines 24783 24787 +4 ==========================================
Hits 14711 14711 - Misses 9191 9195 +4 
Partials 881 881 

- integrate frontend/dockerfile/dockerignore from buildkit
full diff: moby/patternmatcher@v0.5.0...v0.6.0
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
The BuildKit dockerignore package was migrated to the patternmatcher
repository / module. This patch updates our uses of the BuildKit package
with its new location.
A small local change was made to keep the format of the existing error message,
because the "ignorefile" package is slightly more agnostic in that respect
and doesn't include ".dockerignore" in the error message.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
thaJeztah merged commit 41183c2 into docker:masterAug 23, 2023
@thaJeztah
thaJeztah deleted the patternmatcher_dockerignore branch August 23, 2023 11:53
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.

4 participants

@thaJeztah@codecov-commenter@rumpl@vvoland