Skip to content
This repository has been archived by the owner on Mar 5, 2023. It is now read-only.

Fix babel plugin macros use in machines #55

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

danielkcz
Copy link

@danielkcz danielkcz commented Oct 9, 2020

I've noticed there is some attempt in code to make macros working, but no testable example so far. So I've added one and it doesn't seem to work. I have no idea what to do about it. If you can give me hint, I can try to fix it.

image

Edit: Maybe I am misunderstanding what that piece of code does, but simply adding babel-plugin-macros to the list did not solve the problem.

@changeset-bot
Copy link

changeset-bot bot commented Oct 9, 2020

🦋 Changeset detected

Latest commit: 9d50e15

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

This PR includes changesets to release 1 package
Name Type
xstate-codegen 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

@danielkcz
Copy link
Author

@mattpocock This is the last piece (for now) for me to fully convert to codegen. I know I could do without macros, but that's too verbose :) I would appreciate some help, I have no idea what's happening there.

@danielkcz
Copy link
Author

Hah, new year = new brain. I just added babel-plugin-macros to the mix and it solves this. It's ready for merge.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant