-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
removed create appsync fixtures #12572
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
Conversation
@cloutierMat Just checking if this is going to make it to release 4.5. Otherwise, please update the milestone to 4.6 or playground. Thanks. |
a8a2cc0
to
282fc2d
Compare
Test Results - Alternative Providers988 tests 621 ✅ 34m 26s ⏱️ Results for commit 282fc2d. |
Test Results (amd64) - Integration, Bootstrap 5 files 5 suites 2h 21m 47s ⏱️ Results for commit 282fc2d. |
@cloutierMat If this PR needs a milestone bumb again, please do so :) |
@baermat This PR is only removing a fixture that is no longer used. Test suite was executed against ext to validate no tests are relying on it. Can I ask you for a quick review and a merge? 🙏 |
Motivation
This fixture was moved upstream as the underlying resources can't be deployed in the community version of LocalStack
Changes
appsync_create_api
fixtureTesting
External run number: 15783613172 🟢