GitHub
02/16/2023, 5:39 AMStateHandlers type was changed so that it was incompatible with the other StateHandlers type, resulting in conflicts when Typescript merged the two. This caused problems where it was impossible to correctly type StateHandlers (see #1057).
This PR is a breaking change that simply renames the message StateHandlers interface to MessageStateHandlers, and therefore fixes #1057.
Probably the better fix would be to introduce a breaking change that makes both types the same again - but I didn't want to get in to untangling what is supposed to happen.
This must not be squash merged, otherwise you won't get the breaking change in the changelog.
☐ npm run dist works locally (this will run tests, lint and build)
☑︎ Commit messages are ready to go in the changelog (see below for details)
☑︎ PR template filled in (see below for details)
pact-foundation/pact-js
GitHub Actions: build-and-test-windows (18.x)
GitHub Actions: build-and-test-windows (16.x)
GitHub Actions: build-and-test-windows (14.x)
GitHub Actions: build-and-test-ubuntu (18.x)
GitHub Actions: build-and-test-ubuntu (16.x)
GitHub Actions: build-and-test-ubuntu (14.x)
✅ 1 other check has passed
1/7 successful checksGitHub
03/02/2023, 6:53 AM