Skip to content

feat: Check pull requests using Lighthouse CI#96

Merged
smockle merged 8 commits intomainfrom
lighthouse
Dec 10, 2020
Merged

feat: Check pull requests using Lighthouse CI#96
smockle merged 8 commits intomainfrom
lighthouse

Conversation

@smockle
Copy link
Copy Markdown
Contributor

@smockle smockle commented Dec 9, 2020

Lighthouse CI is a CLI version of Lighthouse, which can be used to assess site performance, accessibility, best practices and SEO.

This PR add a Lighthouse CI check to incoming pull requests, so we can automate the kinds of checks @koddsson performed here.

We can set thresholds for specific categories or exclude them. PWA rules are currently excluded.

NB: As configured, results are uploaded to temporary public storage. Results are not sensitive. The same results may be obtained by cloning this public repo and running npx lhci autorun, or by running Lighthouse via the Google Chrome DevTools against the public site.

Results are not sensitive. The same results may be obtained by cloning this public repo and running 'npx lhci autorun', or by running Lighthouse via the Google Chrome DevTools against the public site. Terms: https://github.com/GoogleChrome/lighthouse-ci/blob/master/docs/services-disclaimer.md#temporary-public-storage
@smockle smockle requested a review from a team as a code owner December 9, 2020 20:36
Copy link
Copy Markdown
Contributor

@koddsson koddsson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is neat! I'd love to get the issues fixed and merge this PR.

Comment thread lighthouserc.json
* fix: Lighthouse audit issues

* chore: Skip Lighthouse checks which fail in CI but not DevTools

* chore: Cache Bundler gems to speed-up CI
@smockle smockle merged commit 86ba8bd into main Dec 10, 2020
@smockle smockle deleted the lighthouse branch December 10, 2020 18:09
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.

2 participants