Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(e2e): import from @smithy packages where needed #5012

Merged
merged 1 commit into from
Jul 25, 2023

Conversation

trivikr
Copy link
Member

@trivikr trivikr commented Jul 25, 2023

Issue

Internal JS-4559

Description

Imports from @smithy packages where needed

Testing

All legacy integration tests are successful

$ yarn test:e2e:legacy
...
149 scenarios (149 passed)
492 steps (492 passed)
2m40.959s (executing steps: 2m40.094s)
Done in 163.80s.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@trivikr trivikr changed the title fix(e2e): import from deprecated packages fix(e2e): import from @smithy packages where needed Jul 25, 2023
@trivikr trivikr force-pushed the e2e-legacy-relative-imports branch from a719757 to 5c94ebe Compare July 25, 2023 19:42
@trivikr trivikr marked this pull request as ready for review July 25, 2023 19:44
@trivikr trivikr requested a review from a team as a code owner July 25, 2023 19:44
@trivikr trivikr merged commit 53df336 into aws:main Jul 25, 2023
@github-actions
Copy link

github-actions bot commented Aug 9, 2023

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 Aug 9, 2023
@trivikr trivikr deleted the e2e-legacy-relative-imports branch September 13, 2024 14:18
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