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.

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

Release: Migration of API category plugin #514

Closed
wants to merge 23 commits into from
Closed

Release: Migration of API category plugin #514

wants to merge 23 commits into from

Conversation

phani-srikar
Copy link
Contributor

@phani-srikar phani-srikar commented Jun 6, 2022 •
edited
Loading

Description of changes

This PR releases the following features in main:

  • Migrate API category plugin from Amplify CLI repo
  • Update yarn.lock file to refresh the resolved package versions.
  • Update Github code owners
  • e2e tests update to accommodate for this change in CLI flow feat: diagnose amplify-cli#10383

Issue #, if available

Description of how you validated changes

  • e2e test suite runs successfully
  • Sanity check the API commands using locally built dev binary from setup-dev in a JS app

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.

alharris-at and others added 23 commits May 31, 2022 12:05
@alharris-at
chore: update repo links, jest and gitignore paths post-migration
84cb3bc
@alharris-at
chore: Update Codeowners
7c72052
@alharris-at
chore: use latest amplify-cli and amplify-app packages for testing
7a72aab
@josefaidt @alharris-at
fix: set searchable streaming lambda debug env var default to 0
6a34980
@lazpavel @alharris-at
test (amplify-util-mock, graphql-transformers-e2e-tests): @auth e2e a…
05b2f5f 
…nd velocity exhaustive testing poc
@alharris-at
fix: update cleanup circle contexts, remove unnecessary branches
985e77f
@alharris-at
chore: update function test to point to node 14 (#460)
108d7be
@lazpavel @alharris-at
test (graphql-transformers-e2e-tests): fix @auth delete with partial …
ac2d7d6 
…access test (#25)
@dependabot @alharris-at
build(deps): bump dset from 3.1.1 to 3.1.2 (#465)
3fbbd9b 
Bumps [dset](https://github.com/lukeed/dset) from 3.1.1 to 3.1.2.
- [Release notes](https://github.com/lukeed/dset/releases)
- [Commits](lukeed/dset@v3.1.1...v3.1.2)

---
updated-dependencies:
- dependency-name: dset
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@alharris-at
fix: correct behavior for CreateApiKey config (#463)
45bf5d4
@alharris-at
fix: update cleanup script to correctly use assumed creds in child ac…
a6020df 
…counts (#467)
@alharris-at
chore: clean and rebuild yarn.lock file, update associated code (#473)
9e7a187
@alharris-at
chore: update integration test to clean before setup-dev in order to …
70bc45a 
…mitigate the dup build failures (#475)
@alharris-at
chore: manually clean graphql-transformer-core on each build to avoid…
5126cb5 
… type override issues (#477)
@alharris-at
chore: parallelize account and resource cleanup for e2e testing (#478)
6782d9b
@alharris-at
chore: remove dependency on integration-test for migration tests v5 a…
7504219 
…nd v6 (#479)
@alharris-at
refactor: remove tests that dont apply to our repo space, and remove …
60151b7 
…since we don't have control over it. Consider replacing w/ a different approach (#482)
@alharris-at
chore: split api e2e tests to get runtimes under control, and move ou…
ca9f895 
…t of parent account (#484)
@alharris-at
chore: migrate api_2 test snapshots to api_10 (#485)
9b5af24
@alharris-at
chore: adding resolutions for dependabot alerts (#492)
c4095bd
@AaronZyLee @alharris-at
fix(searchable): change delete and update policy for os instance (#24)
6d4d5ce
@amplify-data-ci
chore(release): Publish [ci skip]
a1d92f4 
 - [email protected]
 - [email protected]
 - [email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - [email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - @aws-amplify/[email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
 - [email protected]
@phani-srikar
release(amplify-category-api): pull latest main after resolving confl…
e5173c5 
…icts
@alharris-at alharris-at marked this pull request as ready for review June 6, 2022 17:57
@alharris-at alharris-at requested a review from a team June 6, 2022 17:57
@alharris-at alharris-at requested a review from a team as a code owner June 6, 2022 17:57
@lgtm-com
Copy link

lgtm-com bot commented Jun 6, 2022

This pull request introduces 4 alerts when merging e5173c5 into a1d92f4 - view on LGTM.com

new alerts:

  • 1 for Useless conditional
  • 1 for Unused variable, import, function or class
  • 1 for Polynomial regular expression used on uncontrolled data
  • 1 for Missing await

@phani-srikar phani-srikar changed the title Pull latest features from main after resolving conflicts Release: Migration of API category plugin Jun 6, 2022
@phani-srikar phani-srikar deleted the resolve-release-main-conflicts branch June 8, 2022 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alharris-at alharris-at alharris-at approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@phani-srikar @alharris-at @josefaidt @lazpavel @AaronZyLee @amplify-data-ci