Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
Moves over to using npm instead of yarn for the repo.
641bbcf - Handles moving all the files and references over
The scripts usage has changed slightly due to differences between npm and yarn (i.e.
yarn examples android
isn't achievable innpm
) but I've tried to replace with dedicated scripts where possible.9d76a74 - Handles the issue that arose due to npm enforcing peer-dependencies, this necessitated a move to newer version of react-testing-library which required the following changes
renderHook
function is now in the main librarywaitForNextUpdate
withwaitFor
from RTL, this required changes to how we were waiting to advance, mostly it just required adding a check to make sure the library had loaded before continuing on. In 2 tests where we test for errors it was handled slightly different.Checklist