Skip to content

Commit 9889bbf

Browse files
committed
squash! copy: add option to strip sparse manifest lists
Clarify deletion log message with curly apostrophe Change "deleting instance from destination manifest" to "deleting instance from destination's manifest" (with typographically correct curly apostrophe) to make it clear we're modifying the destination, not the source. This prevents user confusion about which manifest is being modified and maintains consistent punctuation style. Signed-off-by: Alex Guidi <aguidi@redhat.com>
1 parent 86b3413 commit 9889bbf

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

image/copy/multiple.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ func prepareInstanceOps(list internalManifest.List, instanceDigests []digest.Dig
130130
!slices.Contains(options.Instances, instanceDigest) {
131131
// Create delete operation if we're stripping sparse manifests
132132
if options.SparseManifestListAction == StripSparseManifestList {
133-
logrus.Debugf("deleting instance %s from destination manifest (%d/%d)", instanceDigest, i+1, len(instanceDigests))
133+
logrus.Debugf("deleting instance %s from destination’s manifest (%d/%d)", instanceDigest, i+1, len(instanceDigests))
134134
deleteOps = append(deleteOps, instanceOp{
135135
op: instanceOpDelete,
136136
deleteIndex: i,

0 commit comments

Comments
 (0)