Skip to content

Commit e141c8f

Browse files
authored
Merge pull request #2923 from crazy-max/docs-bake-overrides
chore: comments to not forget to update docs
2 parents 2ee1562 + 0d92ad7 commit e141c8f

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

bake/bake.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -556,6 +556,8 @@ func (c Config) newOverrides(v []string) (map[string]map[string]Override, error)
556556

557557
o := t[kk[1]]
558558

559+
// IMPORTANT: if you add more fields here, do not forget to update
560+
// docs/bake-reference.md and https://docs.docker.com/build/bake/overrides/
559561
switch keys[1] {
560562
case "output", "cache-to", "cache-from", "tags", "platform", "secrets", "ssh", "attest", "entitlements", "network":
561563
if len(parts) == 2 {
@@ -861,6 +863,8 @@ func (t *Target) Merge(t2 *Target) {
861863
}
862864

863865
func (t *Target) AddOverrides(overrides map[string]Override, ent *EntitlementConf) error {
866+
// IMPORTANT: if you add more fields here, do not forget to update
867+
// docs/bake-reference.md and https://docs.docker.com/build/bake/overrides/
864868
for key, o := range overrides {
865869
value := o.Value
866870
keys := strings.SplitN(key, ".", 2)

0 commit comments

Comments
 (0)