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(ui): Bump xstate to ^4.33.0 #2432

Merged
merged 6 commits into from
Aug 9, 2022
Merged

chore(ui): Bump xstate to ^4.33.0 #2432

merged 6 commits into from
Aug 9, 2022

Conversation

wlee221
Copy link
Contributor

@wlee221 wlee221 commented Aug 8, 2022

Description of changes

Bumping xstate to 4.33.0, which contains statelyai/xstate#3126 that fixes #764.

Issue #, if available

#764

Description of how you validated changes

Build and e2e tests passing. This is not really a change in customer's end, because we pointed to latest xstate anyway.

Checklist

  • PR description included
  • yarn test passes
  • Tests are updated

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

@changeset-bot
Copy link

changeset-bot bot commented Aug 8, 2022

🦋 Changeset detected

Latest commit: e66a528

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@aws-amplify/ui Patch
@aws-amplify/ui-react Patch
@aws-amplify/ui-vue Patch
@aws-amplify/ui-angular Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@wlee221 wlee221 temporarily deployed to ci August 8, 2022 20:28 Inactive
@wlee221 wlee221 temporarily deployed to ci August 8, 2022 20:28 Inactive
@wlee221 wlee221 temporarily deployed to ci August 8, 2022 20:28 Inactive
@wlee221 wlee221 temporarily deployed to ci August 8, 2022 20:28 Inactive
@wlee221 wlee221 marked this pull request as ready for review August 9, 2022 04:14
@wlee221 wlee221 requested a review from a team as a code owner August 9, 2022 04:14
@wlee221 wlee221 temporarily deployed to ci August 9, 2022 04:24 Inactive
@wlee221 wlee221 temporarily deployed to ci August 9, 2022 04:24 Inactive
@wlee221 wlee221 temporarily deployed to ci August 9, 2022 04:24 Inactive
@wlee221 wlee221 temporarily deployed to ci August 9, 2022 04:24 Inactive
@wlee221 wlee221 temporarily deployed to ci August 9, 2022 05:07 Inactive
@wlee221 wlee221 temporarily deployed to ci August 9, 2022 05:07 Inactive
@wlee221 wlee221 temporarily deployed to ci August 9, 2022 05:07 Inactive
@wlee221 wlee221 temporarily deployed to ci August 9, 2022 05:07 Inactive
Copy link
Contributor

@slaymance slaymance left a comment

Choose a reason for hiding this comment

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

👍

@@ -42,7 +42,7 @@
"dependencies": {
"lodash": "4.17.21",
"style-dictionary": "3.7.0",
"xstate": "^4.30.6",
"xstate": "^4.33.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

For my knowledge, why not pin the version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good question!

This is to avoid package duplication: Gatsby depends on xstate@^4.x.x, so if we pin the version, customer will get duplicated versions of xstate. And we found that duplicated xstate causes runtime errors.

More details here! #1545

Copy link
Contributor

Choose a reason for hiding this comment

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

Got it, thanks!

@wlee221 wlee221 merged commit b3e6a97 into main Aug 9, 2022
@wlee221 wlee221 deleted the bump-xstate branch August 9, 2022 22:34
@github-actions github-actions bot mentioned this pull request Aug 9, 2022
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.

Warning: No implementation found for action type 'clearFormValues'
5 participants