Skip to content

Conversation

kuhe
Copy link
Contributor

@kuhe kuhe commented Sep 10, 2025

Issue

smithy-lang/smithy-typescript#1600

Description

This is for verifying the tests that will make up part of the specification.

Testing

unit tests.

Checklist

  • [n/a] If the PR is a feature, add integration tests (*.integ.spec.ts).
  • [n/a] If you wrote E2E tests, are they resilient to concurrent I/O?
  • [n/a] If adding new public functions, did you add the @public tag and enable doc generation on the package?

@kuhe kuhe requested a review from a team as a code owner September 10, 2025 19:48
@kuhe kuhe force-pushed the schema/document-testing branch from ddde942 to aff6547 Compare September 10, 2025 19:53
@kuhe kuhe marked this pull request as draft September 10, 2025 19:54
@kuhe
Copy link
Contributor Author

kuhe commented Sep 10, 2025

  • protocol tests, string boxed object bug?

@kuhe kuhe force-pushed the schema/document-testing branch from aff6547 to 88390ca Compare September 10, 2025 20:02
@kuhe kuhe marked this pull request as ready for review September 11, 2025 17:09
@kuhe kuhe force-pushed the schema/document-testing branch 2 times, most recently from 097a8f6 to 7467ac2 Compare September 11, 2025 17:38
@kuhe kuhe force-pushed the schema/document-testing branch from 7467ac2 to ce6f2d7 Compare September 11, 2025 19:16
@trivikr trivikr requested review from trivikr September 11, 2025 20:28
@kuhe kuhe merged commit 9cd9415 into aws:main Sep 12, 2025
2 of 3 checks passed
@kuhe kuhe deleted the schema/document-testing branch September 12, 2025 15:29
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants