Skip to content

Conversation

@krowvin
Copy link

@krowvin krowvin commented Oct 12, 2025

Fixes #561

Solution

Uses gradle to spinup a local server using existing src and then runs the lighthouse CI in a sep task against that temporary service. Writes the output files to disk locally. Will attempt to update/test CI/CD further before submitting. But hoping to

  1. Only run tests if the web-ui files have changed
  2. Block PR if a 508 / performance issue is found (if the score is < 95%)
  3. Load output to artifacts for viewing if a user prefers CI/CD for testing

how you tested the change

Tested on windows/macos

Pull down, run
.\gradlew a11yScan

@krowvin krowvin linked an issue Oct 12, 2025 that may be closed by this pull request
Copy link
Collaborator

@MikeNeilson MikeNeilson left a comment

Choose a reason for hiding this comment

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

generally reasonable, needs a few organization and gradle tweaks which i can pr to this pr for.

@krowvin
Copy link
Author

krowvin commented Oct 14, 2025

The CI/CD is hanging waiting for the server. I was using act locally to try and figure that out.

I also tried to figure out how to dynamically determine URL, as well as, logging in.

Might be able to list files in the web app dir since they are static and run lighthouse on those paths.

For the login, I found a puppet library to add a middleware for login but did not get the chance to actually test.

@sonarqubecloud
Copy link

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.

Lighthouse CI for 508/performance/best practices

3 participants