WPTT / theme-sniffer

Theme Sniffer plugin using sniffs.
MIT License
270 stars 3 forks source link

Allow sniff selection #117

Open timelsass opened 5 years ago

timelsass commented 5 years ago

Feature Request:

It would be handy to have an advanced section which some of these controls might fall under that is tucked away from the main UI. In this section having the ability to select from all the available sniffs per ruleset selected would be a really nice and useful addition. I think the of toggling annotations and warnings was definitely a good thing, and providing a selection of sniffs to run would be the next step for the UI. I assume a theme could just provide it's own custom ruleset and edit it as needed, but that's a bit more complicated for the lesser inclined.

One use case I have had for this is with the textdomain sniffs: I run a process in a framework which fixes my textdomain errors and changes the names and info for my distributed themes. When running tests - I can safely ignore the textdomain errors, but it makes it very hard to sort through and find the other errors.

Another use case I have had for this is when fixing a particular set of errors of one type. I make all changes based on the recommendation, and would like to just run a sniff for that error to ensure I've fixed all issues without having to sort through a list of other errors.

dingo-d commented 5 years ago

Sounds interesting but I'm not sure how would this be done tbh. We should have to list all the rules from the ruleset, make them toggle-able and based on the selection modify the runner. Which sounds a bit complicated 😄

Would definitely like to consider this, but not for the v1.0.0 release (which will hopefully be soon).