Skip to content

fix: align runtime and test compatibility - #369

Open
jterapin wants to merge 1 commit into
serde-opt/6-optimizationsfrom
serde-opt/7-compatibility
Open

jterapin wants to merge 1 commit into
serde-opt/6-optimizationsfrom
serde-opt/7-compatibility

Conversation

@jterapin

@jterapin jterapin commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Applies small runtime, type-signature, and test-setup compatibility fixes on top of #367.

  • Restores RFC 3986 query URI escaping through Smithy::Client::Util.
  • Aligns nullable Schema shape RBS declarations with runtime initialization.
  • Replaces deprecated SimpleCov filter calls in client and XML specs.

Merge order

Merge after #367 and before #366.

Validation

  • Focused client utility spec: 2 examples, 0 failures.
  • Focused XML builder spec: 21 examples, 0 failures.
  • smithy-schema RBS task: 136 examples, 0 failures.
  • RuboCop and git diff --check passed.

@jterapin
jterapin force-pushed the serde-opt/7-compatibility branch from 79d87ea to a616b43 Compare September 14, 2026 19:47
@jterapin
jterapin marked this pull request as ready for review September 14, 2026 20:33
@jterapin
jterapin requested a review from a team as a code owner September 14, 2026 20:33
@jterapin
jterapin added this pull request to stack #370 September 14, 2026 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant