Combine @xstate/graph and @xstate/test
#4896
Conversation
🦋 Changeset detectedLatest commit: 18c84c8 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
|
@Andarist When you get the chance, can you set this up so that it will do a proper 2.0 release? |
|
@Andarist Ping 🏓 |
There was a problem hiding this comment.
I diffed this and the other one in xstate-test. They are very very similar but yet they have some very small differences. I'm confused 😅
| @@ -5,12 +5,7 @@ | |||
| "linked": [], | |||
| "access": "public", | |||
| "baseBranch": "main", | |||
| "ignore": [ | |||
There was a problem hiding this comment.
q: do u plan to release @xstate/graph as a stable release soon? we probably should add @xstate/graph back here and do some cleanup in https://github.com/statelyai/xstate/tree/7c6e2eaa4e5021738efec345b51f59a8e2de662e/.changeset-xstate-test-beta and only then execute the final release
There was a problem hiding this comment.
Yes, I plan to release it soon
| @@ -131,7 +131,7 @@ describe('getShortestPaths', () => { | |||
| todo: 'two' | |||
| } as const | |||
| ], | |||
| filter: (state) => state.context.todos.length < 3 | |||
| stopWhen: (state) => state.context.todos.length >= 3 | |||
There was a problem hiding this comment.
this feels like a breaking change that wasn't mentioned in changesets
There was a problem hiding this comment.
I will add a changeset in a follow-up (clean-up) PR
No description provided.