Juliette

Results 2038 comments of Juliette

> I could see an argument for "do X or Y", but I worry that could cause issues for SA tools that need to figure out which approach someone is...

@Creel I hear what you are saying, but then we get a discussion on when a value should be considered multi-line. Think: ```php $array = [ 'sub-array' => [ 'item1',...

@ernilambar Please read the linked issues before pinging people: https://github.com/WordPress/plugin-check/issues/523#issuecomment-2241002831 As for: > If we could make this array customizable then we can feed our own list of restricted prefixes....

> Instead of patching I‘d just port this sniff over to this repo and modify it to our needs That would be violating both the copyright as well as the...

@swissspidy Quicker iteration ? How ? I suggested 10 days ago to open an issue in the WPCS repo and nobody has even bothered.

Sounds similar to functionality already in the WPCS `PrefixAllGlobals` sniff. Has any of you ever tried running it with the `--report=info` option ?

@davidperezgar `phpcs -ps . --standard=WordPress --sniffs=WordPress.NamingConventions.PrefixAllGobals --report=full,source,info --runtime-set prefixes my_prefix,tgmpa`

> Hello @jrfnl Why is necessary to give prefixes? It isn't supposed to check the number of prefixes instead? If not, how could I check the number of prefixes? Thanks...

@schlndh Thanks for this PR. Some feedback: * Please include what you are trying to fix in the PR description. Your arguments in https://github.com/squizlabs/PHP_CodeSniffer/issues/3556#issuecomment-3393265341 make some sense, but should be...

> I plan to use it as a config option via `` (mainly because of Phpstorm). But IMO having it as a configuration option only seems a bit inflexible. The...