-
Notifications
You must be signed in to change notification settings - Fork 1
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
Improve error messaging #32
base: main
Are you sure you want to change the base?
Conversation
This is actually a duplicate of #26 (which itself is sort of a duplicate of #13) - the blocker has been getting the time to sort out deployments because I believe only @joshmcarthur has access to that. I'll book some time with him when he's back to get this closed off once and for all |
06be9d1
to
e0e2910
Compare
5ecf371
to
446d734
Compare
The performance integration test was failing a lot since i merged #28, it looks like performance doesn't like being the first test run |
Full report: | ||
#{Lighthouse::Matchers.results_directory}/f60a563794dfeedda6feeab0ec4a011c6bd74ff9.json | ||
|
||
To view this report, load this file into https://googlechrome.github.io/lighthouse/viewer/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dang, that's cool✨
2e0406b
to
885818f
Compare
7ae8e2c
to
4c3e415
Compare
4c3e415
to
ee98817
Compare
Provide some visibility into why and how much a score is worse than it was