[chore] Remove visual regression testing for now #1443
Merged
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.
This removes the visual regression testing setup for now. We should prioritize getting it back in at a later point, but as it stands today it doesn't provide much value as the tests are currently failing and seems largely outdated, which also indicates that nobody is really running them on a regular basis. It also creates a lot of delay when initializing a clean checkout of this repo, and when reinstalling node_modules since it triggers a chromium build download which is sometimes incredibly slow.
This PR can be reverted when we figure out a better and more sustainable approach (e.g. a CI environment that can run the tests for us).