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

Upgrade to TypeScript 3.9.7 #1354

Merged
merged 7 commits into from
Aug 4, 2020
Merged

Upgrade to TypeScript 3.9.7 #1354

merged 7 commits into from
Aug 4, 2020

Conversation

davidkpiano
Copy link
Member

This PR does two things:

  1. Upgrades TypeScript to the latest 3.9.7
  2. Simplifies the Action type, based on guidance from Checking is slow in project consuming xstate types microsoft/TypeScript#39826

cc. @amcasey

@changeset-bot
Copy link

changeset-bot bot commented Aug 3, 2020

🦋 Changeset is good to go

Latest commit: ed9a277

We got this.

This PR includes changesets to release 7 packages
Name Type
xstate Minor
@xstate/analytics Major
@xstate/graph Major
@xstate/immer Major
@xstate/react Major
@xstate/test Major
@xstate/vue Major

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

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 3, 2020

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit ed9a277:

Sandbox Source
XState Example Template Configuration
XState React Template Configuration

Copy link

@amcasey amcasey left a comment

Choose a reason for hiding this comment

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

@AlCalzone's original repro LGTM with this change. Thanks for the quick turnaround!

@@ -106,6 +106,7 @@ export function toActionObject<TContext, TEvent extends EventObject>(
const { type, ...other } = action;

actionObject = {
// @ts-ignore
Copy link

Choose a reason for hiding this comment

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

Are you ignoring TS2783: 'type' is specified more than once, so this usage will be overwritten.? I think the error's correct, in which case, you just need to move type after ...exec. If you disagree with the error, I'd love to know more.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, I was trying to figure out what that error was asking for. Thanks!

Copy link

Choose a reason for hiding this comment

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

It looks like a bunch of your other @ts-ignores can also be removed, now that you're on 3.9.7. 😄

Co-authored-by: Mateusz Burzyński <mateuszburzynski@gmail.com>
@davidkpiano davidkpiano merged commit 4c3a274 into master Aug 4, 2020
@davidkpiano davidkpiano deleted the davidkpiano/ts-3.9.7 branch August 4, 2020 11:07
@github-actions github-actions bot mentioned this pull request Aug 4, 2020
@github-actions github-actions bot mentioned this pull request Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants