Denis Žoljom

Results 142 issues of Denis Žoljom

PHP 7.3 relaxed requirements for heredoc and nowdoc syntax. The WPCS native sniffs which test for `T_HEREDOC` and `T_NOWDOC` tokens will need to be reviewed to make sure that they...

Focus: Modern PHP

From what I've looked at this sniff, the `private $tab_width` is `The --tab-width CLI value that is being used.` I tried changing the default tab width in my ruleset to...

Type: Bug

Following the previous issue, we should also think about remaining `master` branch to something other like `main`. Suggestions are welcomed for the naming ofc. Some references: https://dev.to/_garybell/why-github-renaming-master-is-good-4b64/ https://www.zdnet.com/article/github-to-replace-master-with-alternative-term-to-avoid-slavery-references/ https://www.cnet.com/news/microsofts-github-is-removing-coding-terms-like-master-and-slave/

Meta: Repo

As discussed in #1908 we should scan the code for any problematic and non-inclusive words (`whitelist/blacklist` for instance). The principles discussed in https://github.com/PHPCSStandards/PHPCSExtra/issues/59 should be taken into the account. I...

Type: Enhancement
Status: help wanted
Status: good first issue

## Bug Report **Describe the current, buggy behavior** On my WSL2 I have a local setup (I'm using valet-wsl package to handle my websites), and for php version handling I'm...

This PR is the result of me writing an issue seeing if this is solved or not, then realizing I should dig a bit into the codebase, and after digging...

I'm rewriting my plugin and trying to use patterns like factory and service locator like here: https://github.com/schlessera/wcbtn-2018-api ([factory](https://github.com/schlessera/wcbtn-2018-api/blob/master/src/PluginFactory.php), [plugin](https://github.com/schlessera/wcbtn-2018-api/blob/master/src/Plugin.php)). I have set up my testing, loaded everything using composer (I'm...

The title of the chapter `Plugin Unit Tests` is a bit misleading. Unit tests are usually done when adding/modifying some core feature/component, and plugins work within WordPress so the tests...

scope:documentation

This PR will remove unnecessary files for production release (psalm, configuration files, etc.)

I'm wondering what is the correct way of handling dynamic properties defined in the `beforeEach` method? In the documentation there is the following example: ```php beforeEach(function () { $this->hey =...

bug