Skip to content

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

removed create appsync fixtures #12572

wants to merge 1 commit into from

Conversation

cloutierMat
Copy link
Contributor

@cloutierMat cloutierMat commented May 1, 2025 •
edited
Loading

Motivation

This fixture was moved upstream as the underlying resources can't be deployed in the community version of LocalStack

Changes

  • Delete appsync_create_api fixture

Testing

External run number: 15783613172 🟢

@cloutierMat cloutierMat added semver: patch Non-breaking changes which can be included in patch releases review: merge when ready Signals to the reviewer that a PR can be merged if accepted labels May 1, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented May 1, 2025 •
edited
Loading

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 42m 22s ⏱️ - 1m 3s
4 890 tests ±0  4 116 ✅ ±0  774 💤 ±0  0 ❌ ±0 
4 892 runs  ±0  4 116 ✅ ±0  776 💤 ±0  0 ❌ ±0 

Results for commit 282fc2d. ± Comparison against base commit 6348947.

♻️ This comment has been updated with latest results.

@cloutierMat cloutierMat added this to the 4.5 milestone May 6, 2025
@anisaoshafi
Copy link
Contributor

@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.

@cloutierMat cloutierMat modified the milestones: 4.5, 4.6 Jun 4, 2025
@cloutierMat cloutierMat force-pushed the remove-unused-fixture branch from a8a2cc0 to 282fc2d Compare June 20, 2025 16:39
@github-actions GitHub Actions
Copy link

Test Results - Preflight, Unit

21 635 tests  ±0   19 978 ✅ ±0   6m 11s ⏱️ -3s
     1 suites ±0    1 657 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 282fc2d. ± Comparison against base commit 6348947.

@github-actions GitHub Actions
Copy link

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 7s ⏱️ -2s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 282fc2d. ± Comparison against base commit 6348947.

@github-actions GitHub Actions
Copy link

Test Results - Alternative Providers

988 tests   621 ✅  34m 26s ⏱️
  4 suites  367 💤
  4 files      0 ❌

Results for commit 282fc2d.

@github-actions GitHub Actions
Copy link

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 21m 47s ⏱️
5 247 tests 4 321 ✅ 926 💤 0 ❌
5 253 runs  4 321 ✅ 932 💤 0 ❌

Results for commit 282fc2d.

@cloutierMat cloutierMat marked this pull request as ready for review June 20, 2025 18:54
@baermat
Copy link
Member

baermat commented Jul 1, 2025

@cloutierMat If this PR needs a milestone bumb again, please do so :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dominikschubert dominikschubert Awaiting requested review from dominikschubert dominikschubert is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
review: merge when ready Signals to the reviewer that a PR can be merged if accepted semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Milestone
4.6
Development

Successfully merging this pull request may close these issues.
3 participants
@cloutierMat @anisaoshafi @baermat