Skip to content
This repository has been archived by the owner on Oct 8, 2021. It is now read-only.

update to the latest major version of @testing-library/dom #5

Merged
merged 1 commit into from Jul 1, 2021

Conversation

JakeChampion
Copy link
Contributor

@JakeChampion JakeChampion commented Jun 28, 2021

We are not using any of the features which were removed or changed in the new version.

The new version also removes the need for the hack.js file we needed to use. The reason we needed that hack.js file was to work around an issue in a dependency of testing-library/dom called pretty-format. pretty-format's new version has fixed the issue

@JakeChampion JakeChampion temporarily deployed to o-autocomple-test-elfdswslxgxq June 28, 2021 15:24 Inactive
@origamiserviceuser origamiserviceuser added this to incoming in Origami ✨ Jun 28, 2021
@github-actions github-actions bot added the component Relates to an Origami component label Jun 28, 2021
@JakeChampion JakeChampion temporarily deployed to o-autocomple-test-elfdswslxgxq June 30, 2021 11:14 Inactive
We are not using any of the features which were removed or changed in the new version.

The new version also removes the need for the hack.js file we needed to use this is because, the package is using the latest major versio of pretty-format, which contains this update jestjs/jest#11569
@JakeChampion JakeChampion temporarily deployed to o-autocomple-test-elfdswslxgxq June 30, 2021 11:17 Inactive
@JakeChampion JakeChampion marked this pull request as ready for review June 30, 2021 11:24
@JakeChampion JakeChampion requested a review from a team as a code owner June 30, 2021 11:24
@JakeChampion JakeChampion merged commit 99aadbe into alphagov Jul 1, 2021
Origami ✨ automation moved this from incoming to complete Jul 1, 2021
@JakeChampion JakeChampion deleted the test branch July 1, 2021 11:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component Relates to an Origami component
Projects
Origami ✨
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant