Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Lighthouse CI integration #96

Open
localnerve opened this issue Jul 10, 2016 · 3 comments
Open

Lighthouse CI integration #96

localnerve opened this issue Jul 10, 2016 · 3 comments

Comments

@localnerve
Copy link
Owner

Integrate lighthouse into CI and fail build on threshold problems.

@addyosmani
Copy link

Do you have a feel for how well the current version in master does when tested against Lighthouse? For React projects in particular, you might find it useful to test against the branch with TTI support over here too: GoogleChrome/lighthouse#450

@localnerve
Copy link
Owner Author

localnerve commented Jul 13, 2016

I ran it about a month ago (~6/12) and fixed everything at that time until it was 100%.

Updates

For anyone reading this later, items Lighthouse pointed out to me were extremely insightful/useful - stuff I probably wouldn't have caught until much later:

  1. Allows non-https requests (A CloudFlare config update for this project)
  2. WCAG contrast issues

A super tool so far - THANK YOU (all).

I can't wait to be able to check this automatically when I push changes to master. That will really be great. Not so sure about setup for Travis though - In the past, Chrome has been a little tricky to setup and maintain.

Any examples of setting up Lighthouse for CI (Chrome setup, etc)?

Anyways, thanks for the info, I'll be sure to checkout the TTI support branch.

@localnerve
Copy link
Owner Author

localnerve commented Jul 16, 2016

@addyosmani Is the TTI branch supposed to be runnable using the instructions + "develop instructions section" from GoogleChrome/lighthouse readme? I ran into a dep issue.
Repeat of this: GoogleChrome/lighthouse#503

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants