[wip] Add Lighthouse in CI

[bollwyvl]

Thanks again for this theme!

This adds some more GitHub Actions to run Lighthouse against the built artifacts, as a start to #61, and should help inform performance and SEO as well:

• https://github.com/foo-software/lighthouse-check-action#usage-standard-example
• https://github.com/foo-software/lighthouse-check-status-action#usage-standard-example

This needs an upstream PR to get merged on larsoner/circleci-artifacts-redirector-action, so that we can use the circle URL in other steps. I've temporarily pointed it at my branch.

I've also added a few quick-win role and aria-labels to the navs, and done a little bit of looking into some of the things I determined from running local Lighthouse.

[bollwyvl]

Looks like it's a timescale thing: the status job is always getting run from master, so it does not include these changes, and pull_request won't have the right information to work with circle. hm....

[hoetmaaiers]

Is this update also available in the Github Checks part (in this PR)?

[bollwyvl]

No, it doesn't work (yet) because the status event is decoupled from the pr cycle... It needs some crucial bits from the circle webhook payload.

I think, if merged, it would show up, just like the redirector, but it's hard to say.

[hoetmaaiers]

Ok, not to strange indeed.
The addition of Lighthouse in any case receives a +1 from me.

[bollwyvl]

Cool, the upstream pr got merged/released. Perhaps I do a smaller PR chain that: - just runs/publishes the report - add the score thresholds without any changes - adds the few small fixes, updates the thresholds

The threshold changes wouldn't be applied until after merge, but the alternatives (running headless chrome with lighthouse installed) seem... Hard, and less representative of a production host (http/2, ssl).

[choldgraf]

I am a +1 on anything that gives us iterative improvements without adding much complexity to our build process, or maintenance burden :-)

[choldgraf]

I think that this one is no longer needed, since we now do have lighthouse checks in our integration tests. I'll close it, but please re-open or let me know if this was adding a different functionality as well!

[bollwyvl]

Yup, don't know why this is still open, close away