Exclude node_modules and vendor from theme check plugin

Categories: Development, Theme, WordPress.org

While I created countless themes, I’m currently creating my first theme for the WordPress.org theme repo. To ensure a smooth reviewing process, I’m using the plugin Theme Check as mentioned in the theme review handbook.

After adding both composer and npm to my theme, I ended up with an Fatal error: Allowed memory size of 536870912 bytes exhausted error. Investigating this issue brought me to:

https://github.com/WordPress/theme-check/issues/4https://github.com/WordPress/theme-check/pull/9https://github.com/WordPress/theme-check/issues/103

I found the solution to my specific issue in:

https://github.com/WordPress/theme-check/pull/9/commits/64402b1586bee4a87a0afdf7dfee913958068168

Thus, I went ahead, opened the main.php in the root folder of the plugin and replaced

View this gist on GitHub

with

View this gist on GitHub

While this might not be the best solution to solve this issue, it allows me to analyse my theme.

Patchwork #46099

Categories: WordPress.org

Recently, the Twenty Nineteen theme had been moved from Github to Trac. As Twenty Nineteen is the first core theme that uses npm and therefore contains a node_modules folder, running grunt resulted in an error. I just created a minor patch, to solve this issue, and to make WordPress yet a little bit better. 😀
The Trac ticket can be found on Ignore theme based node_modules and my patch can be viewed via 46099.diff.

Patchwork #45912

Categories: WordPress.org

“Great things are done by a series of small things brought together.”
This quote of Vincent Van Gogh suits perfectly to my core contributions. Even though they’re not big, yet they ensure to make WordPress even better. Having said that, I just submitted a small patch to ensure that <hr>‘s are shown in full-width instead of in Gutenberg’s shortened version.
The Trac issue can be viewed on Twenty Nineteen: Horizontal rule is very narrow and my patch can be viewed via 45912.diff.