Skip to content

Match DefaultStringInterpolation appendInterpolation overloads#469

Merged
weissi merged 4 commits into
apple:mainfrom
kukushechkin:match-defaultstringinterpolation-overloads
May 28, 2026
Merged

Match DefaultStringInterpolation appendInterpolation overloads#469
weissi merged 4 commits into
apple:mainfrom
kukushechkin:match-defaultstringinterpolation-overloads

Conversation

@kukushechkin

Copy link
Copy Markdown
Contributor

Custom MetadataValue.StringInterpolation broke compatibility with DefaultStringInterpolation. Rearrange appendInterpolation overloads to match DefaultStringInterpolation to maintain backward compatibility.

Motivation:

Custom MetadataValue.StringInterpolation broke compatibility with DefaultStringInterpolation. Rearrange appendInterpolation overloads to match DefaultStringInterpolation to maintain backward compatibility.

Modifications:

MetadataValue.StringInterpolation appendInterpolation overloads changed.

Result:

Code relying on matching correct DefaultStringInterpolation appendInterpolation overload compiles.

@kukushechkin kukushechkin added the 🔨 semver/patch No public API change. label May 27, 2026
@kukushechkin
kukushechkin marked this pull request as ready for review May 27, 2026 16:14
@kukushechkin
kukushechkin requested a review from FranzBusch May 27, 2026 16:14

@FranzBusch FranzBusch left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we please add some tests for all of those

Comment thread Tests/LoggingTests/MetadataValueInterpolationTest.swift Outdated
Comment thread Tests/LoggingTests/MetadataValueInterpolationTest.swift Outdated
@kukushechkin
kukushechkin requested a review from FranzBusch May 28, 2026 11:56
@weissi
weissi merged commit 2aed77a into apple:main May 28, 2026
52 of 53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 semver/patch No public API change.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants