-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
add custom audit recipe #2255
add custom audit recipe #2255
Conversation
e6ad766
to
d6c75f7
Compare
d6c75f7
to
bd00d2f
Compare
'use strict'; | ||
|
||
module.exports = { | ||
// Run our tests along with all the default Lighthouse tests. |
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.
"Run your custom tests...."
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.
done
'searchable-audit' | ||
], | ||
|
||
// Create a new 'My site metrics' section in the default report for our results. |
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.
How about a 1., 2., 3., 4. on these.
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.
done
docs/recipes/custom-audit/readme.md
Outdated
|
||
This Lighthouse [gatherer](searchable-gatherer.js)/[audit](searchable-audit.js) pair will take that value from the context of the page and test whether or not it stays below a test threshold. | ||
|
||
The config file tells Lighthouse where to find the gatherer and audit files, when to run them, and how to incoporate their output into the Lighthouse report. |
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.
incorporate
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.
done
I didn't get a chance to review, but I was coming here to give it a thumbs up. 🌮 |
adds a custom gatherer/audit to our recipes. Happy to bikeshed on all of this :)
No rush on landing this, but we'll be using a version of this at I/O, so get in comments if you want to influence what we show there :)