You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@webhintio/contributors I've been giving this some thought and had a slightly different idea. What about letting the site choose the configuration instead?
E.g. the online scanner could check for a .hintrc file at the root of the origin and use that (perhaps first looking for one in the same path as the current page).
This provides persistence and consistency across a team as multiple users may choose to run webhint against the same URL. We could even pick it up by default from other entry points like the CLI and browser extension (still allowing it to be overridden by the user of course).
I think that will add some extra complexity (and possible vector attacks) that I'm not sure will be that valuable.
Some thoughts:
What if the .hintrc they upload points to hints that aren't available in the online scanner
That file might be used for abuse, we will need to check that files aren't gigantic, malware, and such. The configuration is done early in the process and we will have to be extra careful
Some hints might not be available (e.g.: image optimization requires user/pass)
What are the users of the online scanner? Initially the online scanner is for people to have an idea of what the tool is capable of and what type of things it can analyze. It shouldn't be a replacement for the CLI (or at least it wasn't the initial purpose)
I'm more inclined to allow some basic configuration like the one in the extension as @bmt-systems proposed in webhintio/online-service#676. My reasons are:
The online scanner is a "lets see if this tool is useful" kind of thing . Once people start using a .hintrc probably use it in the CLI, CI/CD, etc. and hopefully have gates to prevent "issues" from reaching production
We will have similar configuration experiences in the website and the extension. The barrier to install an extension is a lot lower than the CLI and if they are already familiarized with how the configuration works even better. Having to write a .hintrc file might be asking too much for configs
Part of the frontend code is already there, we should implement the server side code for it.
The text was updated successfully, but these errors were encountered: