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: Migrate CI/CD to GitHub Actions #11638

Merged
merged 72 commits into from
Aug 3, 2023

Conversation

stocaaro
Copy link
Member

@stocaaro stocaaro commented Jul 17, 2023

Description of changes

This is the current state of the Github Actions CI/CD migration. It models the following automated processes:

  • PRs run
    • unit tests
    • bundle size
  • Merging to main runs:
    • unit tests
    • bundle size
    • integ/e2e tests
    • When all pass
      • Release to unstable preid
  • Merging to configured preid branch runs:
    • unit tests
    • bundle size
    • integ/e2e tests
    • When all pass
      • Release to specified preid
  • Merging to release branch runs:
    • unit tests
    • bundle size
    • integ/e2e tests
    • When all pass
      • Release to latest
      • Updates required-release tag if needed
      • Commits new versions and docs updates
      • Merged updates back to main branch

Patterns follows:

  • Triggered workflows prefixed by the event type that triggers them.
  • Callable workflow used to encapsulate reusable operations that run on isolated Github runners.
  • Actions used to encapsulate code called across multiple workflows.
  • All internal actions/workflows and actions referenced by relative path action and workflow.
  • Use caching to reduce runtime of processes
  • Prefer to test by execution - including NPM push, which is protected using verdaccio

Examples executions (for execution speed, I only ran a single RN test and ran all other tests on a single browser):

Common actions

Description of how you validated changes

By executing workflows, either trimmed to get to the desired condition promptly, but always rolling the whole process together and running end to end.

Checklist

  • PR description included
  • yarn test passes

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

stocaaro and others added 30 commits June 23, 2023 08:58
* feat(e2e): add integ test runner and test

* fix(e2e): retain existing workflows

* fix(e2e): remove old e2e example

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>
* chore: add e2e browser auth tests

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>
* chore: add e2e integ_react_predictions

* chore: add browser e2e interactions

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>
Co-authored-by: Aaron S <94858815+stocaaro@users.noreply.github.com>
* prod & dev

* -n 3

* turn off fail-fast

---------

Co-authored-by: Aaron S <94858815+stocaaro@users.noreply.github.com>
Co-authored-by: Sridhar <ashwsrir@amazon.com>
Co-authored-by: Sridhar <ashwsrir@amazon.com>
Co-authored-by: Sridhar <ashwsrir@amazon.com>
* chore: Re-arrange integ test configurations and inprove preflight

* Fix is-preflight flag

* Change preflight to prebuild for consistencu

* Disable fast fail on unit tests
Co-authored-by: Sridhar <ashwsrir@amazon.com>
Co-authored-by: Sridhar <ashwsrir@amazon.com>
* chore: test upload artifacts

* chore: test uploading artifact
Co-authored-by: Sridhar <ashwsrir@amazon.com>
* chore: add deploy workflow

* chore: gh actions publish code clean up

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>
@@ -0,0 +1,18 @@
name: Push - release from main to unstable

Choose a reason for hiding this comment

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

nit: dk if possible to combine this with push-latest-release and use the branch name as a variable, but ... also sometimes duplicating code is easier.

Copy link
Member Author

Choose a reason for hiding this comment

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

I get what you mean. This is closer the the pre-id workflow. For clarity, I have the different entry point workflows separated with duplicate code pushed down into callable workflows where the small amount of redundant calling adds clear entry points for users to trace forward from.

If there where problems with overlapping runs because of complex selection logic or some other driver that made a single workflow for multiple purposes better, I could get behind that. In this case, I like how simple each of these entry points are and am intentionally tuning for that.

@stocaaro stocaaro marked this pull request as ready for review August 2, 2023 01:19
@stocaaro stocaaro requested review from a team as code owners August 2, 2023 01:19
@codecov-commenter
Copy link

codecov-commenter commented Aug 2, 2023

Codecov Report

Merging #11638 (972c2f3) into main (f54e7c9) will not change coverage.
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           @@
##             main   #11638   +/-   ##
=======================================
  Coverage   83.71%   83.71%           
=======================================
  Files         347      347           
  Lines       21684    21684           
  Branches     4613     4613           
=======================================
  Hits        18152    18152           
  Misses       3245     3245           
  Partials      287      287           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

"test:size": "lerna run test:size --no-bail",
"test:duplicates": "./scripts/duplicates-yarn.sh",
"test:license": "license-check-and-add check -f license_config.json",
"test:github-actions": "node ./scripts/test-github-actions.js",
Copy link
Member Author

Choose a reason for hiding this comment

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

As part of the security review, we have identified certain GH Action best practices that should be statically checked to ensure we keep our changes in line with the guidance / bar we have set.

This command is a static check for the practices identified.

  • We should never directly embed template variables ${{ var }} in run blocks.
  • We should always provide fully qualified uses references for external actions

jimblanc
jimblanc previously approved these changes Aug 3, 2023
Copy link
Contributor

@jimblanc jimblanc left a comment

Choose a reason for hiding this comment

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

Great work, some nits below but looks good 🔥

The yarn.lock file! 🤩

.github/actions/npm-publish/action.yml Outdated Show resolved Hide resolved
.github/actions/npm-publish/action.yml Outdated Show resolved Hide resolved
.github/actions/setup-samples-staging/action.yml Outdated Show resolved Hide resolved
.github/actions/setup-samples-staging/action.yml Outdated Show resolved Hide resolved
.github/workflows/callable-bundle-size-tests.yml Outdated Show resolved Hide resolved
.github/workflows/callable-npm-publish-release.yml Outdated Show resolved Hide resolved
.github/workflows/callable-prebuild-samples-staging.yml Outdated Show resolved Hide resolved
Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com>
@@ -17,7 +17,7 @@ jobs:
e2e:
secrets: inherit
uses: ./.github/workflows/callable-release-verification.yml
unstable-release:
next-release:
Copy link
Contributor

Choose a reason for hiding this comment

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

Good catch

Copy link
Member

@HuiSF HuiSF left a comment

Choose a reason for hiding this comment

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

🚀

@stocaaro stocaaro merged commit 956be8c into aws-amplify:main Aug 3, 2023
28 checks passed
@stocaaro stocaaro mentioned this pull request Aug 3, 2023
3 tasks
AllanZhengYP pushed a commit to AllanZhengYP/amplify-js that referenced this pull request Aug 4, 2023
* feat: Actions for pr testing (#12)

* feat: Add working e2e example (#14)

* chore: Add configurable e2e tests to GH workflows (#15)

* feat(e2e): add integ test runner and test

* fix(e2e): retain existing workflows

* fix(e2e): remove old e2e example

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: add e2e browser auth tests (#16)

* chore: add e2e browser auth tests

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: add e2e browser predictions interactions tests (#17)

* chore: add e2e integ_react_predictions

* chore: add browser e2e interactions

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>
Co-authored-by: Aaron S <94858815+stocaaro@users.noreply.github.com>

* chore: limit PR concurrency (#19)

* chore: add build_type and enable retry (#18)

* prod & dev

* -n 3

* turn off fail-fast

---------

Co-authored-by: Aaron S <94858815+stocaaro@users.noreply.github.com>

* chore: add e2e browser datastore tests (#20)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: add e2e browser api,storage tests (#21)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: fix single browser provided in integ-config (#22)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Run dev and prod on the same runner

* chore: Add detox testing (#26)

* Feat/GitHub actions/test ashwinkumar6 (#27)

* chore: Re-arrange integ test configurations and improve prebuild (#29)

* chore: Re-arrange integ test configurations and inprove preflight

* Fix is-preflight flag

* Change preflight to prebuild for consistencu

* Disable fast fail on unit tests

* feat: Change yaml load output pattern

* chore: fix e2e integ_datastore_auth_v2 test

* chore: disable playwright tests (#33)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: add google env variables for e2e datastore tests (#35)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: add support for uploading artifacts  (#34)

* chore: test upload artifacts

* chore: test uploading artifact

* chore: Detox fix and step cleanup

* chore: Increase detox time limit

* chore: Turn warn to ignore for upload artifact misses

* chore: Disable broken detox tests

* chore: Disable PN testing and remove datastore test not running in circle

* chore: e2e add configurable time_out and retry_count (#36)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Increase deviceTracking timeout

* chore: add deploy workflow (#37)

* chore: add deploy workflow

* chore: gh actions publish code clean up

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Model full CI/CD flow - improve preid release (#38)

* fix: Cache on the single yarn.lock instead of all

* fix: Sync with main

* chore: Revert to relative path use (#40)

* chore: Revert to relative path use

* fix: Remove path from workflows

* chore: Add post-release merge to main

* chore: Remove orphaned workflow files

* chore: e2e add integ_auth_test_cypress_no_ui test (#39)

* chore: e2e add integ_auth_test_cypress_no_ui test

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: change github name and email

* chore: update yarn.log

* chore: move additional e2e commented tests (#43)

* chore: move additional e2e commented tests

* chore: add required permissions to script file

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: e2e keep test_name same as CCI (#45)

* chore: e2e keep test_name same as CCI
* chore: e2e re-arrange integ config
---------
Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: update actions typo

Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com>

* chore: update actions typo

Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com>

* chore: cleanup github_email and github_name (#46)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: cleanup load verdaccio action (#47)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: actions code cleanup (#48)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Remove unused actions and use env vars instead of injection in run commands (#49)

* chore: add retention period and timeout for cache (#50)

* chore: add retention period and timeout for cache

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Chanke js token name

* chore: Filter preid content

* chore: Move file names around and consume env vars

* fix: Rest bundle size increase

* chore: Compactify on syntax

* chore: dynamically fetch the repo_owner_name (#52)

* chore: dynamically fetch the repo_owner_name

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Configure staging to use corresponding branch name

* chore: Naming and cleanup

* chore: Naming and dependency workflow improvements

* chore: Fix vars use for gh user/email

* chore: run e2e Rollup + DataStore test on just chrome (#54)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Add preid protection (#57)

* chore: Only publish to npm from the aws-amplify org

* chore: add concurrency to push workflows (#56)

* chore: add concurrency to push workflows

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Prepare for merge to main

* chore: Fix circle build

* chore: Okay, this will fix circle!

* chore: Integrate license checking into pr check

* chore: Add github action static tests

* chore: Fix test command

* Update .github/actions/npm-publish/action.yml

Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com>

* chore: Fix names and descriptions

---------

Co-authored-by: Ashwin Kumar <ashwinkumar2468@gmail.com>
Co-authored-by: Sridhar <ashwsrir@amazon.com>
Co-authored-by: Lucas Yingshyan Ku <99565133+yingku@users.noreply.github.com>
Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com>
Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com>
svidgen added a commit that referenced this pull request Aug 10, 2023
* feat(data): add CCI config for GraphQL API optimistic UI E2E tests (#11585)

* chore: update size limit that is too tight (#11652)

* chore: update size limit that is too tight

* chore(prediction): update size limit after custom s3 client

* fix: duplicate entry in connection states (#11579)

Co-authored-by: Aaron S <94858815+stocaaro@users.noreply.github.com>
Co-authored-by: AllanZhengYP <zheallan@amazon.com>

* fix(api-rest): refactor ajax method to not relying on side effects (#11498)

* fix(api-rest): refactor ajax method to not relying on side effects

the previous Signer.sign() implementation not only returns signed
request, but also sets the signed headers to input request. The
recent refactor makes it side-effect-less.

However this change breaks RestClient.ajax() clock skew correction
which relies on the x-amz-date header set by signer to the input
request object to indicate the current client side time.

this fix resolves #11480

* chore: Enable automatic license checking (#11604)

* chore(release): Publish [ci skip]

 - @aws-amplify/analytics@6.4.1
 - @aws-amplify/api-graphql@3.4.5
 - @aws-amplify/api-rest@3.4.0
 - @aws-amplify/api@5.3.5
 - @aws-amplify/auth@5.5.5
 - aws-amplify@5.3.5
 - @aws-amplify/cache@5.1.5
 - @aws-amplify/core@5.7.0
 - @aws-amplify/datastore-storage-adapter@2.0.42
 - @aws-amplify/datastore@4.6.5
 - @aws-amplify/geo@2.2.0
 - @aws-amplify/interactions@5.2.5
 - @aws-amplify/notifications@1.5.0
 - @aws-amplify/predictions@5.4.5
 - @aws-amplify/pubsub@5.4.1
 - @aws-amplify/pushnotification@5.0.39
 - @aws-amplify/storage@5.8.0

* chore(release): update API docs [ci skip]

* chore: Move Storage TS dependency to `devDependencies` (#11666)

* chore: pin @smtihy/types to 1.1.0 to prevent build failure in TS3.8 (#11673)

chore: ping @smtihy/types to 1.1.0 to prevent build failure in TS3.8

This is ok because the AWS SDK and crypto dependencies have been pinned
already

* chore: remove smithy resolution (#11682)

* feat: custom user agent Geo changes for UI handoff (#11632)

* feat: custom user agent Storage changes for UI handoff (#11627)

* chore: move storage code to InternalStorage.ts

* chore: update class and module name to InternalStorage and update imports

* feat: add customUserAgentDetails param to public methods

* feat: add customUserAgentDetails to getProperties methods, fix cancel overload

* feat: make Storage extend internal storage and override public methods

* feat: export InternalStorage from internals scope

* feat: send user agent details from public copy api to service call

* fix: add missing storage action, fix other minor issues

* feat: add userAgentValue with received user agent details to AWSS3Provider options

* fix: send getproperties instead of copy from getproperties method

* test: update tests to handle new form of passing user agent

* Revert "test: update tests to handle new form of passing user agent"

This reverts commit 4bc7c48.

* fix: fix jumbled StorageActions

* test: update CustomUserAgent test

* test: remove storageAction param and asserts from test

* chore: increase bundle size limits

* chore: publish feature branch for testing

* Feat/custom user agent UI/inappmessaging (#11656)

* chore: copy InAppMessaging to InternalInAppMessaging

* feat: add customUserAgentDetails param to InternalInAppMessaging

* feat: make InAppMessaging extend InternalInAppMessaging

* feat: send sync userAgentDetails through to service call

* fix: add SyncMessages value to InAppMessagingAction

* chore: remove extra parameter from non-service-call functions

* fix: fix typo in InAppMessagingAction

* feat: send user agent details through identify user to service call

* chore: remove unused import

* feat: allow InAppMessaging to be configured independently of Notifications

* test: update tests for internal changes

* build: export InternalInAppMessaging from scoped path

* fix: resolve bundle size

* chore: bundle size settings

* chore: increase bundle size limits

* chore: fix typo

* fix: update references to type with typo

* Feat/custom user agent UI/geo (#11657)

* chore: copy Geo code into InternalGeo

* feat: add geo actions

* feat: add customUserAgentDetails to geo provider

* feat: add customUserAgentDetails to public api's and send to service calls

* feat: make Geo extend InternalGeo and override public API's

* build: export InternalGeo from scoped geo/internals path

* chore: increase bundle size limits

* chore: increase bundle size limits

* feat: add user agent param to action that was missed

* chore: update doc-strings with new param

* Revert "Merge branch 'feat/custom-user-agent-ui/main' into feat/custom-user-agent-ui/storage"

This reverts commit f0e8657, reversing
changes made to ec1ed1a.

* fix: remove userAgentValue from CommonStorageOptions

* feat: add userAgentValue param to StorageProvider and implement

* fix: update config type to param type at position

* test: update test class based on changes

* chore: add new param to jsdocs

* chore: update comment from LastParameter (now ConfigParameter)

* chore: add comment explaining StorageProviderApiOptionsIndexMap

* chore: increase bundle size limits

* fix: send userAgentValue through storage put and list

* chore: remove outdated TODO comment

* chore: increase bundle size limits

* feat: custom user agent InAppMessaging changes for UI handoff (#11639)

* chore: copy InAppMessaging to InternalInAppMessaging

* feat: add customUserAgentDetails param to InternalInAppMessaging

* feat: make InAppMessaging extend InternalInAppMessaging

* feat: send sync userAgentDetails through to service call

* fix: add SyncMessages value to InAppMessagingAction

* chore: remove extra parameter from non-service-call functions

* fix: fix typo in InAppMessagingAction

* feat: send user agent details through identify user to service call

* chore: remove unused import

* feat: allow InAppMessaging to be configured independently of Notifications

* test: update tests for internal changes

* build: export InternalInAppMessaging from scoped path

* fix: resolve bundle size

* chore: bundle size settings

* chore: increase bundle size limits

* chore: fix typo

* fix: update references to type with typo

* chore: update jsdocs in InternalInAppMessaging

* chore: increase bundle size limit

* chore: shorten jsdocs comment

* fix: first attempt to fix InAppMessaging internals

* fix: fix internals package.json paths

* test: fix import form internals

* chore: increase bundle size limits

* chore(release): Publish [ci skip]

 - @aws-amplify/analytics@6.5.0
 - @aws-amplify/api-graphql@3.4.6
 - @aws-amplify/api-rest@3.5.0
 - @aws-amplify/api@5.4.0
 - @aws-amplify/auth@5.6.0
 - aws-amplify@5.3.6
 - @aws-amplify/cache@5.1.6
 - @aws-amplify/core@5.8.0
 - @aws-amplify/datastore-storage-adapter@2.0.43
 - @aws-amplify/datastore@4.7.0
 - @aws-amplify/geo@2.3.0
 - @aws-amplify/interactions@5.2.6
 - @aws-amplify/notifications@1.6.0
 - @aws-amplify/predictions@5.5.0
 - @aws-amplify/pubsub@5.5.0
 - @aws-amplify/pushnotification@5.0.40
 - @aws-amplify/storage@5.9.0

* chore(release): update API docs [ci skip]

* chore: Migrate CI/CD to GitHub Actions (#11638)

* feat: Actions for pr testing (#12)

* feat: Add working e2e example (#14)

* chore: Add configurable e2e tests to GH workflows (#15)

* feat(e2e): add integ test runner and test

* fix(e2e): retain existing workflows

* fix(e2e): remove old e2e example

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: add e2e browser auth tests (#16)

* chore: add e2e browser auth tests

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: add e2e browser predictions interactions tests (#17)

* chore: add e2e integ_react_predictions

* chore: add browser e2e interactions

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>
Co-authored-by: Aaron S <94858815+stocaaro@users.noreply.github.com>

* chore: limit PR concurrency (#19)

* chore: add build_type and enable retry (#18)

* prod & dev

* -n 3

* turn off fail-fast

---------

Co-authored-by: Aaron S <94858815+stocaaro@users.noreply.github.com>

* chore: add e2e browser datastore tests (#20)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: add e2e browser api,storage tests (#21)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: fix single browser provided in integ-config (#22)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Run dev and prod on the same runner

* chore: Add detox testing (#26)

* Feat/GitHub actions/test ashwinkumar6 (#27)

* chore: Re-arrange integ test configurations and improve prebuild (#29)

* chore: Re-arrange integ test configurations and inprove preflight

* Fix is-preflight flag

* Change preflight to prebuild for consistencu

* Disable fast fail on unit tests

* feat: Change yaml load output pattern

* chore: fix e2e integ_datastore_auth_v2 test

* chore: disable playwright tests (#33)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: add google env variables for e2e datastore tests (#35)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: add support for uploading artifacts  (#34)

* chore: test upload artifacts

* chore: test uploading artifact

* chore: Detox fix and step cleanup

* chore: Increase detox time limit

* chore: Turn warn to ignore for upload artifact misses

* chore: Disable broken detox tests

* chore: Disable PN testing and remove datastore test not running in circle

* chore: e2e add configurable time_out and retry_count (#36)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Increase deviceTracking timeout

* chore: add deploy workflow (#37)

* chore: add deploy workflow

* chore: gh actions publish code clean up

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Model full CI/CD flow - improve preid release (#38)

* fix: Cache on the single yarn.lock instead of all

* fix: Sync with main

* chore: Revert to relative path use (#40)

* chore: Revert to relative path use

* fix: Remove path from workflows

* chore: Add post-release merge to main

* chore: Remove orphaned workflow files

* chore: e2e add integ_auth_test_cypress_no_ui test (#39)

* chore: e2e add integ_auth_test_cypress_no_ui test

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: change github name and email

* chore: update yarn.log

* chore: move additional e2e commented tests (#43)

* chore: move additional e2e commented tests

* chore: add required permissions to script file

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: e2e keep test_name same as CCI (#45)

* chore: e2e keep test_name same as CCI
* chore: e2e re-arrange integ config
---------
Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: update actions typo

Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com>

* chore: update actions typo

Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com>

* chore: cleanup github_email and github_name (#46)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: cleanup load verdaccio action (#47)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: actions code cleanup (#48)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Remove unused actions and use env vars instead of injection in run commands (#49)

* chore: add retention period and timeout for cache (#50)

* chore: add retention period and timeout for cache

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Chanke js token name

* chore: Filter preid content

* chore: Move file names around and consume env vars

* fix: Rest bundle size increase

* chore: Compactify on syntax

* chore: dynamically fetch the repo_owner_name (#52)

* chore: dynamically fetch the repo_owner_name

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Configure staging to use corresponding branch name

* chore: Naming and cleanup

* chore: Naming and dependency workflow improvements

* chore: Fix vars use for gh user/email

* chore: run e2e Rollup + DataStore test on just chrome (#54)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Add preid protection (#57)

* chore: Only publish to npm from the aws-amplify org

* chore: add concurrency to push workflows (#56)

* chore: add concurrency to push workflows

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: Prepare for merge to main

* chore: Fix circle build

* chore: Okay, this will fix circle!

* chore: Integrate license checking into pr check

* chore: Add github action static tests

* chore: Fix test command

* Update .github/actions/npm-publish/action.yml

Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com>

* chore: Fix names and descriptions

---------

Co-authored-by: Ashwin Kumar <ashwinkumar2468@gmail.com>
Co-authored-by: Sridhar <ashwsrir@amazon.com>
Co-authored-by: Lucas Yingshyan Ku <99565133+yingku@users.noreply.github.com>
Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com>
Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com>

* chore: Migrate from circle to github actions (#11714)

* chore: Disable RSC test for firefox (#11719)

* chore: Reintroduce circle without deploy/post-release steps (#11720)

* chore: Reintroduce circle without deploy/post-release steps

* chore: Fix yarn.lock issue

* chore: Update yarn.lock (#11725)

* chore: add canary tests (#11721)

* chore: add canary testing

---------

Co-authored-by: Aaron S <94858815+stocaaro@users.noreply.github.com>
Co-authored-by: Francisco Rodriguez <frodriguez.cs@gmail.com>

* fix: update on-schedule-canary-test.yml (#11733)

Update on-schedule-canary-test.yml

* chore: Remove jsdoc (#11731)

* chore: remove unused cypress (#11732)

* chore: update react-native version (#11734)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* feat: custom user agent InternalCognitoUser (#11709)

* chore: port code from CognitoUser and CognitoUserPool to internals

* chore: delete InternalCognitoUserPool - separate PR

* chore: update imports and class name

* feat: add userAgentValue param to cognito client api's

* feat: add userAgentValue param to InternalCognitoUser api's, pass to client

* feat: make CognitoUser extend InternalCognitoUser and override api's

* test: update tests to work with additional param, minor fix

* chore: clean up unused imports and const's

* feat: add missing userAgentValue param

* chore: add new param to documentation comments

* build: export InternalCognitoUser from internals scope

* fix: remove default from InternalCognitoUser to fix scoped export

* fix: fix typo causing signOut failure

* build: correct index.d.ts files with changes

* test: revert auth tests

* chore: increase bundle size limits

* chore: simplify InternalCognitoUser import

---------

Co-authored-by: israx <70438514+israx@users.noreply.github.com>

* chore: run canary everyday at 9am (#11735)

* chore: run canary everyday at 9am

* chore: code cleanup

Co-authored-by: Venkata Ramyasri Kota <34170013+kvramyasri7@users.noreply.github.com>

* chore: update canary cron to 9amPDT

* chore: code cleanup

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>
Co-authored-by: Venkata Ramyasri Kota <34170013+kvramyasri7@users.noreply.github.com>

* feat: custom user agent Auth changes for UI handoff (#11606)

* chore: copy Auth code to InternalAuth

* feat: add customUserAgentDetails param to InternalAuth methods

* feat: make AuthClass extend InternalAuthClass, override functions with extra param

* build: export auth internals from /internals scope

* fix: changes signUp back to method type instead of prop so Auth can extend the method

* chore: update auth lint settings to allow class arrow functions to end in semi-colon

* fix: add overloaded method signature back in - unrelated change

* test: fix auth tests by mocking InternalAuthClass instead of AuthClass

* chore: change arrow methods to normal functions

* fix: remove async from overridden async functions

* chore: increase bundle size limits

* fix: remove internalSignUp and fix rest attr with overload and any

* fix: remove internalSignUp and fix rest attr with overload and any

* chore: update jsdocs

* chore: increase bundle size limits

* chore: Added retries to unstable DataStore canaries (#11740)

* fix(notifications): ios module pod cannot be autolinked in RN version < 0.69 (#11751)

* chore: Remove eslint from `amazon-cognito-identity-js` (#11749)

* feat: custom user agent InternalCognitoUserPool (#11716)

* chore: port CognitoUserPool over to InternalCognitoUserPool

* chore: restore CognitoUserPool

* feat: add userAgentValue parameter to signUp

* feat: update class name and imports

* feat: make CognitoUserPool extend internal and override signUp

* feat: export InternalCognitoUserPool from internals index

* feat: send userAgentValue to client service call

* test: fix internalsIndex test

* build: export InternalCognitoUserPool from internals scope

* chore: update CognitoUserPool type to extend internal

* Apply suggestions from code review

Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com>

* chore: remove unnecessary test on internals index

* chore: increase bundle size limit

---------

Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com>
Co-authored-by: israx <70438514+israx@users.noreply.github.com>

* chore: Bump DataStore join unit test time limit (#11753)

* chore: Upgrade amazon-cognito-identity-json Webpack & other changes (#11746)

* fix: Remove cpx dependency from pubsub (#11752)

* chore: update increase run count for unstable integ tests (#11758)

Co-authored-by: Sridhar <ashwsrir@amazon.com>

* chore: bump commonmarker from 0.23.9 to 0.23.10 in /docs (#11754)

Bumps [commonmarker](https://github.com/gjtorikian/commonmarker) from 0.23.9 to 0.23.10.
- [Release notes](https://github.com/gjtorikian/commonmarker/releases)
- [Changelog](https://github.com/gjtorikian/commonmarker/blob/v0.23.10/CHANGELOG.md)
- [Commits](gjtorikian/commonmarker@v0.23.9...v0.23.10)

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

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com>

* chore: Update `amazon-cognito-identity-js` to use Webpack 5 (#11757)

* fixed build issues; revert api-graphql package merge changes

* fixed bundle size limits, added empty test command to types package to keep lerna happy

* bundle size limits

* updated preid for api-v6

* fixed branch name for tagged release trigger

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: David McAfee <mcafd@amazon.com>
Co-authored-by: AllanZhengYP <zheallan@amazon.com>
Co-authored-by: takpen <131546291+takpen@users.noreply.github.com>
Co-authored-by: Aaron S <94858815+stocaaro@users.noreply.github.com>
Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com>
Co-authored-by: aws-amplify-bot <aws@amazon.com>
Co-authored-by: israx <70438514+israx@users.noreply.github.com>
Co-authored-by: erinleigh90 <106691284+erinleigh90@users.noreply.github.com>
Co-authored-by: Ashwin Kumar <ashwinkumar2468@gmail.com>
Co-authored-by: Sridhar <ashwsrir@amazon.com>
Co-authored-by: Lucas Yingshyan Ku <99565133+yingku@users.noreply.github.com>
Co-authored-by: William Lee <43682783+wlee221@users.noreply.github.com>
Co-authored-by: Francisco Rodriguez <frodriguez.cs@gmail.com>
Co-authored-by: Hui Zhao <10602282+HuiSF@users.noreply.github.com>
Co-authored-by: Venkata Ramyasri Kota <34170013+kvramyasri7@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

8 participants