Denis Žoljom
Denis Žoljom
#### What are you trying to achieve? I'd like to change the test descriptions so that they are a bit more... well, descriptive 😅 [@testdoc annotation](https://docs.phpunit.de/en/9.6/annotations.html#testdox) #### What do you...
As it currently is the following code ```php echo ClassName::someMethodName( $attributes, $test ); ``` Would trigger the EscapeOutput sniff for the class name, but also for any attributes of the...
This PR will address the issue of handling mixed case function/method/class names when those are passed as custom-allowed properties in certain sniffs. For instance, if we pass a custom sanitization...
In the coding standards only the naming of the classes is covered [here](https://make.wordpress.org/core/handbook/best-practices/coding-standards/php/#naming-conventions). So a class called `Example_Class` has to reside in a `class-example-class.php` file. What should we use for...
I noticed that in the docblocks we are unsing FQCN when adding types to the parameters. We are also importing those at the top of the file. In most of...
The current version constraint is preventing installing the lib on PHP 8.1 (what I'm currently working on). Did you consider raising the PHP version support? What would be the necessary...
I was creating a workflow that included a job with a WordPress container, and when I used `actions/checkout@v2` I noticed that none of my `.` files (except `.gitattributes` and `.gitignore`)...
It would be a good feature to have, when you don't look where you are in the terminal and you accidentally run the command in the plugin folder or somewhere...
Currently, the `eightshiftLibsRepo` is a hardcoded value, which makes installation of different forks for testing purposes, a bit harder (you need to manually change repos). Would it be possible to...
### Describe your bug When you run the installation on PHP 8.1 you don't get any error out. It just doesn't run. There should be a clear error on what...