Skip to content

Commit f2eda73

Browse files
aledbfclaude
andcommitted
refactor(imagemeta): leave the merged containerEnv order to PropertyOrder
Every reader of MergedConfig.ContainerEnvOrder goes through features.PropertyOrder, which already keeps a key's first position, so deduplicating it in MergeConfiguration had no effect any test could see. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent b707e94 commit f2eda73

1 file changed

Lines changed: 0 additions & 1 deletion

File tree

‎internal/imagemeta/merge.go‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,6 @@ func MergeConfiguration(entries []Entry) *MergedConfig {
134134
}
135135
}
136136

137-
m.ContainerEnvOrder = features.KeyOrder(m.ContainerEnvOrder)
138137
m.Mounts = dedupeMountsByTarget(m.Mounts)
139138
m.ForwardPorts = dedupeForwardPorts(m.ForwardPorts)
140139
m.HostRequirements = mergeHostRequirements(entries)

0 commit comments

Comments
 (0)