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

Release Commit #759

Merged
merged 6 commits into from
Aug 23, 2022
Merged

Release Commit #759

merged 6 commits into from
Aug 23, 2022

Conversation

alharris-at
Copy link
Contributor

Description of changes

Merging from main to release.

Issue #, if available

N/A

Description of how you validated changes

E2E tests run

Checklist

  • PR description included
  • yarn test passes
  • Tests are changed or added
  • Relevant documentation is changed or added (and PR referenced)
  • New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

sundersc and others added 6 commits August 19, 2022 12:17
This reverts commit 9a070c6.

Revert "Remove test enforcing bugged behavior"

This reverts commit 3fa19b2.

Revert "Fix feature flag usage"

This reverts commit 4a60931.

Revert "Remove unneeded FF prefix"

This reverts commit 2d7e37b.

Revert "Correct some linting mistakes"

This reverts commit 4e80008.

Revert "Fix value overwrite in unit tests"

This reverts commit 52de86c.

Revert "Tests: add tests for directive config merge"

This reverts commit 8b83c8c.

Revert "Allow public subscriptions to be created"

This reverts commit eba6acd.

Revert "Feature: enable deep merge default directive config with FF"

This reverts commit b1fdf59.
chore(graphql): subscription rtf unused filter types cleanup
test(graphql): add subscription tests with variable syntax
@alharris-at alharris-at requested a review from a team as a code owner August 23, 2022 19:53
@lgtm-com
Copy link

lgtm-com bot commented Aug 23, 2022

This pull request introduces 1 alert when merging cdced31 into bbfc243 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

Copy link
Contributor

@sundersc sundersc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alharris-at alharris-at merged commit 3550082 into release Aug 23, 2022
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.

3 participants