Denis Žoljom
Denis Žoljom
Ready for the final review @jrfnl 🙂
Just a check: would this handle things like ```php namespace Custom_Namespace; class My_Class { public function get_posts() { $args = array( 'post_type' => 'post', 'post_status' => 'publish', 'perm' => 'readable',...
> There is currently no sniff available in WPCS which checks that WP native classes when used in a namespaced context are prefixed with a \ or have a use...
Will this be used on `.php` files only, or on `.js` files as well? Because Gutenberg editor is using [babel-plugin-makepot](https://github.com/WordPress/gutenberg/tree/master/packages/babel-plugin-makepot) for translation generation in the `.js` files. I'm not sure...
> The plan is to provide a different config file for plugins with this part in it How come? They are pretty much the same, and just adding a check...
This should be fixed after new version if I'm not mistaken.
Would it make more sense that the path should always have `Blocks/components` part in it? Because we're always rendering components, no? All the components are located in that folder. I...
Just blanket removing would really introduce a security issue, so for now, try to test if the projects will work if you replace the current regex with `Blocks\/components` and see...
Maybe a check could be made to see if the component name is located in the `components` path, independently of the `$parentPath` variable? That way you could never fetch a...
After the new version, I'm not sure this is applicable 🤷🏼♂️