WPTT / theme-sniffer

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

Check for required files #115

Closed timelsass closed 5 years ago

timelsass commented 5 years ago

Feature Request:

Since the theme doesn't have to be the active theme, I think we should introduce a preliminary scan of required files from the files array to provide some error feedback of missing and required files.

Some that come to mind: functions.php, style.css, readme.txt, (license{.txt|.md|})

dingo-d commented 5 years ago

The theme needs to have style.css, otherwise it will be broken (won't be shown in the list to sniff and will throw a notice in the Appearance screen). We could do a search for the minimum necessary files, this sounds reasonable (looping through all the files isn't a taxing operation in php) 🙂

dingo-d commented 5 years ago

This has been merged to feature/use-namespace.