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

fix(core): bundle react-native-url-polyfill to unblock jest test failure #11422

Merged
merged 4 commits into from
May 27, 2023

Conversation

AllanZhengYP
Copy link
Member

Description of changes

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)

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

@AllanZhengYP AllanZhengYP requested review from a team as code owners May 27, 2023 00:02
@codecov-commenter
Copy link

codecov-commenter commented May 27, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.00%. Comparing base (eeb6c9c) to head (e8a9d43).
Report is 1719 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #11422   +/-   ##
=======================================
  Coverage   83.00%   83.00%           
=======================================
  Files         240      240           
  Lines       20170    20170           
  Branches     4356     4356           
=======================================
  Hits        16743    16743           
  Misses       3140     3140           
  Partials      287      287           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AllanZhengYP AllanZhengYP merged commit 8137ee7 into main May 27, 2023
erinleigh90 added a commit that referenced this pull request May 29, 2023
* chore(release): Publish [ci skip]

 - aws-amplify@5.2.3
 - @aws-amplify/datastore-storage-adapter@2.0.33
 - @aws-amplify/datastore@4.4.0
 - @aws-amplify/predictions@5.2.0
 - @aws-amplify/storage@5.3.0

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

* fix(notifications): Add babel plugins to devDependencies (#11414)

* fix(storage): make getProperties param optional (#11413)

* Revert "fix(storage): make getProperties param optional (#11413)"

This reverts commit 5d00b1f.

* Revert "feat(storage): getProperties API (#11378)"

This reverts commit 3bed12b.

* fix(core): bundle react-native-url-polyfill to unblock jest test failure (#11422)

* fix(core): bundle react-native-url-polyfill to unblock jest test failure

* Revert "test(notification): unblock unit test failure of rn url polyfill"

This reverts commit dbacebc.

* chore: remove unused code

* chore: remove unused polyfills/URL/package.json

* chore(release): Publish [ci skip]

 - @aws-amplify/analytics@6.1.1
 - @aws-amplify/api-graphql@3.2.1
 - @aws-amplify/api-rest@3.1.1
 - @aws-amplify/api@5.1.1
 - @aws-amplify/auth@5.3.7
 - aws-amplify@5.2.4
 - @aws-amplify/cache@5.0.33
 - @aws-amplify/core@5.3.1
 - @aws-amplify/datastore-storage-adapter@2.0.34
 - @aws-amplify/datastore@4.4.1
 - @aws-amplify/geo@2.0.33
 - @aws-amplify/interactions@5.0.33
 - @aws-amplify/notifications@1.1.7
 - @aws-amplify/predictions@5.2.1
 - @aws-amplify/pubsub@5.1.16
 - @aws-amplify/pushnotification@5.0.33
 - @aws-amplify/storage@5.3.1

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

---------

Co-authored-by: israx <70438514+israx@users.noreply.github.com>
Co-authored-by: aws-amplify-bot <aws@amazon.com>
Co-authored-by: Chris F <5827964+cshfang@users.noreply.github.com>
Co-authored-by: Jim Blanchard <jablanch@amazon.com>
Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com>
Co-authored-by: AllanZhengYP <zheallan@amazon.com>
@AllanZhengYP AllanZhengYP deleted the fix/bundle-polyfill branch May 30, 2023 00:34
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.

4 participants