Andrey Savchenko

icon indicating a website link https://www.Rarst.net

icon location Kyiv, Ukraine icon location I believe in a Web of content sites that are a delight to discover, read, and navigate.

Results 76 comments of Andrey Savchenko

Warn against lambda callbacks in add_filter() and add_action()

I've opened a trac ticket with suggestion to improve the API: [Improve identifying of non–trivial callbacks in hooks](https://core.trac.wordpress.org/ticket/46635)

Warn against lambda callbacks in add_filter() and add_action()

> The facts are that closures in WP actions and filters break the current API. Ideally the coding standards should warn the developers that they are doing something wrong. So...

Newline required after opening brace triggered for allowed loop syntax

I disagree about this being specific to loop or needing an exception for it. In my opinion rule that is specifically about _braces_ formatting is irrelevant to alternative syntax. Alternative...

Code style should enforce documentation for actions and filters

Seems to be blocked by no upstream implementation in WPCS still.

require() & include()

Could you please elaborate what do you mean by "map" here?

require() & include()

I don't think we had anything like prototyped in DevHub context, but makes sense as future enhancement. How to treat files is still in the air though, since most of...

Parse array parameter docs

Actually the parameters are _not_ markdown list, but more of a case of special formatting. Not sure what is it based on / called properly. :) ``` * @param array...

Parse array parameter docs

So, should we wait on it for DevHub? Is there estimate for PSR-5 finalization (thinking aloud, I will go look myself tomorrow, about to go off for the night) ?...

Parse array parameter docs

I... don't remember. :) Production reference does run some hacks (I had never seen myself) on top of Parser, but I doubt it's anything as extensive as dealing with those...

Get rid of sleep pauses in process

Personally I am yet to see something break with quick run. And yet to hear someone coherently remember and explain _which_ race condition this is supposedly handling. :)