-
Notifications
You must be signed in to change notification settings - Fork 824
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
chore: dev cb merge #12744
chore: dev cb merge #12744
Conversation
* API plugin version bump to v5.3.0 (aws-amplify#12642) * chore: update API plugin dependencies to v5.3.0 * chore: re-add codegen in bump script * fix: updated readSchema API * fix: update codegen ui version and add processFile util function (aws-amplify#12670) Co-authored-by: Justin Shih <[email protected]> * fix: version bump (aws-amplify#12680) * fix: version bump (aws-amplify#12684) * chore(release): Publish latest [ci skip] - @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] - @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] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - [email protected] - @aws-amplify/[email protected] - [email protected] - @aws-amplify/[email protected] - [email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - [email protected] - [email protected] - [email protected] - @aws-amplify/[email protected] - [email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - [email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] * fix: version bump * chore(release): Publish latest [ci skip] - @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] - @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] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - [email protected] - @aws-amplify/[email protected] - [email protected] - @aws-amplify/[email protected] - [email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - [email protected] - [email protected] - [email protected] - @aws-amplify/[email protected] - [email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] - [email protected] - @aws-amplify/[email protected] - @aws-amplify/[email protected] --------- Co-authored-by: Phani Srikar Edupuganti <[email protected]> Co-authored-by: Justin Shih <[email protected]> Co-authored-by: Justin Shih <[email protected]> Co-authored-by: aws-amplify-bot <[email protected]>
* ci: draft codeowners for codebuild * ci: fix autocorrect * Update .github/CODEOWNERS to include admins Co-authored-by: Kamil Sobol <[email protected]> * ci: use only admins list --------- Co-authored-by: Kamil Sobol <[email protected]>
* chore: migrate to yarn 2 (aws-amplify#12501)
…#12715) * chore: provide better mitigation steps for hosting error * chore: improve error message for publishing apps * chore: improve error messaging * chore: add quotes * chore: address codeql warning * chore: revert
…plify#12623) * feat: throw message when adding lambda function with same name * feat: generalQuestionsWalkthrough test
* feat: display error message instead of object * Update packages/amplify-util-mock/src/func/index.ts Co-authored-by: John Hockett <[email protected]> * feat: update printer and fix tests * fix: api-extract * fix: nested statements --------- Co-authored-by: John Hockett <[email protected]>
…-generation fix: skip automatic form generation when datastore is not enabled
# registy URL update changes .yarnrc.yml file | ||
git update-index --assume-unchanged .yarnrc.yml | ||
fi | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I copied this over from the changes made to publish.sh
"cloud-e2e-local": "source ./scripts/cloud-e2e.sh && cloudE2ELocal", | ||
"cloud-e2e-cb": "bash -c 'source ./scripts/cloud-e2e.sh && cloudE2E'", | ||
"cloud-e2e-cb-beta": "bash -c 'source ./scripts/cloud-e2e.sh && cloudE2EBeta'", | ||
"cloud-e2e-cb-local": "bash -c 'source ./scripts/cloud-e2e.sh && cloudE2ELocal'", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated this to match the new bash source syntax
"test-ci": "nx run-many --parallel=4 --target=test --all --exclude='@aws-amplify/amplify-provider-awscloudformation' -- --ci --runInBand && nx run-many --target=test --projects='@aws-amplify/amplify-provider-awscloudformation' -- -ci --runInBand", | ||
"test": "nx run-many --target=test --all -- --runInBand", | ||
"test-changed": "lerna run test --since dev", | ||
"test-ci": "lerna run test --concurrency 4 -- --ci -i", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we may need to revisit this and implement the exclusion logic for the amplify-provider-awscloudformation test again, but this time with Lerna
@@ -421,7 +410,7 @@ function _integrationTest { | |||
|
|||
echo "Configuring Amplify CLI" | |||
yarn rm-dev-link && yarn link-dev && yarn rm-aa-dev-link && yarn link-aa-dev | |||
export PATH=$(yarn global bin):$PATH | |||
export PATH=$(pwd)/.bin:$PATH |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this also might break the integration test and need revisited
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. @@ Coverage Diff @@
## cb #12744 +/- ##
=====================================
Coverage ? 48.44%
=====================================
Files ? 841
Lines ? 38062
Branches ? 7749
=====================================
Hits ? 18439
Misses ? 18036
Partials ? 1587 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Description of changes
Strategy:
Passing CI run:
https://app.circleci.com/pipelines/github/aws-amplify/amplify-cli/20741/workflows/4eb78493-5e87-49dc-a461-08c55dbaf585
Issue #, if available
Description of how you validated changes
Checklist
yarn test
passesBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.