Kyle

Results 316 comments of Kyle

Note: the [src/conf/phpmd.schema.json](https://github.com/phpmd/phpmd/pull/1058/files#diff-c12b9f17eecc79ee8d087fb1e04ad2d47b08e3a2d01f0e0eb461841cbb4f7f35) file is what allow auto-completion in IDEs: ![image](https://github.com/phpmd/phpmd/assets/5966783/37027ec1-ab84-4930-8a9b-88803e96f0d5) To be automatically applied to `phpmd.yml` files by IDE, we'll need to submit it in https://github.com/SchemaStore/schemastore/tree/master/src/schemas/json Then to be...

[src/test/resources/files/rulesets/phpmd.yml](https://github.com/phpmd/phpmd/pull/1058/files#diff-c905627dbfa1cb2f2c5260b69a58a38358009eba24e9b7e21f6d64afe3612d73) is an example of how it would look to configure rule with a YAML file. It would work the same by providing JSON file or PHP file returning the...

In theory bcmul can go even above 64 bits, but I wonder if it has a real interest in practice for npath (and also why it would be the only...

> Lots of things going on in one PR, most of it is nice, but a bit hard to review and comment on individual things with it all there. I'm...

I finished implementation. But I suspect users might also need some pattern/regexp exception such as `test*` to mean that name of method starting by "test" can be any length.

So I got carried away a bit again and made a pattern system for exceptions property. As I'm already a bit frustrated by the inconsistency of how we handle exceptions...

Fix for https://github.com/predis/predis/pull/1347#issuecomment-1691308594

Trimmed the whitespace eclint complained about: https://github.com/predis/predis/actions/runs/5967543294/job/16189493022?pr=1370#step:6:17

Test added. You need to "Approve and run workflows" to trigger tests and get result from `coveralls`

Redis 7 errors unrelated (might have been introduced by last week update of Redis 7.2) Can be verified with #1372