Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author Samuel Wood (Otto)

    (@otto42)

    www.ads-software.com Admin

    No, those checks should indeed cause errors and failures.

    Theme-check is primarily developed for use here on www.ads-software.com, because all themes uploaded to the directory are run through it at the time of that upload. If it skipped those, then you would be able to include those directories in released versions of themes, which you obviously should not do.

    More likely is that I’d add a check to cause a hard failure on a theme containing those directories.

    Thread Starter Remco Tolsma

    (@remcotolsma)

    Ok, sounds good. But still it would be nice if you could improve the listdir() function. An /node_modules/ folder can grow quickly over 150+ MB with more then 30.000 files. With that many files the theme check feature will take forever. It took me quit some time to figure out that the large /node_modules/ folder was causing this issue. I guess more users could have this issue. If the listdir() function can ignore the /bower_components/**/* and /node_modules/**/* files that would be great. You can then still add a hard failure on existance of the folders /bower_components/ and /node_modules/.

    Plugin Author Samuel Wood (Otto)

    (@otto42)

    www.ads-software.com Admin

    I develop theme check on GitHub. Feel free to help me out, if you like.

    https://github.com/Otto42/theme-check

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Theme development with Bower, Node.js, etc.’ is closed to new replies.