Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test visual regressions #28

Merged
merged 10 commits into from
Apr 3, 2024
Merged

Test visual regressions #28

merged 10 commits into from
Apr 3, 2024

Conversation

john-kurkowski
Copy link
Owner

@john-kurkowski john-kurkowski commented Mar 29, 2024

Test visual regressions with Lost Pixel, a 3rd party service with a GitHub Action. Lost Pixel is a cheaper, open-core Percy clone.

Draft

  • Fix GitHub Actions running forever
  • Fix GitHub Actions ERR_CONNECTION_REFUSED
    • Had to expose Gatsby's development server beyond just localhost
  • Test all routes in this site

@john-kurkowski
Copy link
Owner Author

Lost Pixel's page snapshots seem to hold up GitHub Actions forever. They complete locally via npx lost-pixel local just fine. Converting this PR to draft for now.

@john-kurkowski john-kurkowski changed the title Add Lost Pixel checks Test visual regressions Mar 30, 2024
@john-kurkowski john-kurkowski marked this pull request as ready for review April 3, 2024 23:02
@john-kurkowski john-kurkowski merged commit 413a501 into master Apr 3, 2024
9 checks passed
john-kurkowski added a commit that referenced this pull request Apr 3, 2024
* Copy Lost Pixel getting started instructions
* Bump actions versions
* Test Gatsby production build, instead of development server
* Copy npm scripts from newer upstream Gatsby starter
* Allow visiting production build via IP
* List all routes for testing
* Upgrade ECMAScript version, for newer string matching functions
@john-kurkowski john-kurkowski deleted the lost-pixel branch April 3, 2024 23:16
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.

1 participant