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

chore: dev cb merge #12744

Merged
merged 23 commits into from
Jun 6, 2023
Merged

chore: dev cb merge #12744

merged 23 commits into from
Jun 6, 2023

Conversation

awsluja
Copy link
Contributor

@awsluja awsluja commented Jun 5, 2023

Description of changes

Strategy:

  1. checkout cb
  2. create branch from cb "dev-cb-merge-jun-3"
  3. merge dev
  4. resolve conflicts

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

  • 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
  • Pull request labels are added

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

jhockett and others added 18 commits May 19, 2023 11:32
* 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]>
…#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
@awsluja awsluja requested review from a team as code owners June 5, 2023 22:33
# registy URL update changes .yarnrc.yml file
git update-index --assume-unchanged .yarnrc.yml
fi

Copy link
Contributor Author

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'",
Copy link
Contributor Author

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",
Copy link
Contributor Author

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
Copy link
Contributor Author

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-commenter
Copy link

codecov-commenter commented Jun 5, 2023

Codecov Report

❗ No coverage uploaded for pull request base (cb@c1aeac9). Click here to learn what that means.
The diff coverage is n/a.

❗ 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

@awsluja awsluja merged commit fcfebe1 into aws-amplify:cb Jun 6, 2023
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.