Skip to content

Conversation

@ntucker
Copy link
Collaborator

@ntucker ntucker commented Aug 13, 2019

Optional peerDeps have been added to reflect the needed testing library versions. These are optional because you don't have to use the testing part of rest-hooks.

Future work: 'rest-hooks/test' should be its own package so breaking changes can be separated.

@ntucker ntucker force-pushed the react-hooks-test-upgrade branch from ab5f0d1 to 7f40977 Compare August 13, 2019 23:37
@ntucker ntucker changed the title [breaking][pkg] Upgrade to @testing-library/react-hooks [breaking][pkg] Upgrade to @testing-library packages Aug 13, 2019
@ntucker ntucker force-pushed the react-hooks-test-upgrade branch from 85c9633 to 50f5551 Compare August 14, 2019 01:20
@ntucker ntucker added this to the 3.0 milestone Aug 14, 2019
@ntucker ntucker requested a review from zacharyfmarion August 14, 2019 01:26
@ntucker ntucker force-pushed the react-hooks-test-upgrade branch from 50f5551 to 023ff45 Compare August 14, 2019 01:29
unstable_scheduleCallback,
// eslint-disable-next-line @typescript-eslint/camelcase
unstable_NormalPriority,
} from 'scheduler/unstable_mock';
Copy link
Collaborator Author

@ntucker ntucker Aug 14, 2019

Choose a reason for hiding this comment

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

all testing with scheduler is done via this mock module rather than jest time mocking

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@ntucker
Copy link
Collaborator Author

ntucker commented Aug 14, 2019

build currently failing because coveralls is down

@ntucker ntucker requested review from mprobber and nickcherry August 14, 2019 01:34
@ntucker ntucker changed the title [breaking][pkg] Upgrade to @testing-library packages [pkg] Upgrade to @testing-library packages Aug 14, 2019
@@ -0,0 +1,124 @@
---
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

these new files are just because they didn't change from 2.0 previously so it didn't create a copy

@ntucker ntucker removed this from the 3.0 milestone Aug 14, 2019
@ntucker ntucker merged commit 8fbfb7a into master Aug 15, 2019
@ntucker ntucker deleted the react-hooks-test-upgrade branch August 15, 2019 00:48
Copy link

@nickcherry nickcherry left a comment

Choose a reason for hiding this comment

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

👍

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.

5 participants