-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Feature: score + api #22
Comments
This is such a good idea. Thing is, though, there are certain rules that don't apply to everyone, e.g. Google Analytics. How do we count this toward the score? |
You only count the rules that are enabled in the configuration. |
Yeah but GA is enabled by default. At least currently.
…On Mon, Mar 6, 2017 at 1:29 PM, Barry vd. Heuvel ***@***.***> wrote:
You only count the rules that are enabled in the configuration.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#22 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AHrt0mKHBDWxiFTMocvgz6g1zwG8KOxnks5ri6evgaJpZM4MR7Wb>
.
|
Maybe you could assign it a LEVEL error notice, so it doesn't cost points... |
Instead of the percentage, we could use an 'impact' value, like Google Pagespeed Insights does. Passed: 0 (no impact), Notice: 1, Warning:10, Error: 25, Critical: 100 (or something) Or you could say that a Notice is optional, and doesn't have impact. Or create an 'Optional' rule. Then instead of the percentage, add the 'impact' you could have by following all the rules. In some cases you could thinkg about specifying an impact directly in the Rule, if you want to be more specific. Then the impact can also be used to sort the rules. |
It would be nice to have some sort of 'score' to the results and a total score. For example, passed rule = 10 points, notice 9, warning 7, error 4, critical 0. Then sort the results in order of lowest score (most critical on top) and calculate the total score percentage to give the website a score.
That score could be also used to automate it a bit for an Api, eg call it with a list of rules, use a down to check them periodically, integrate with other services etc.
The text was updated successfully, but these errors were encountered: