Ian Dunn

I think that's a good idea, but I doubt I'd ever have time to implement it myself. Finding time to review/tweak a PR is difficult, too, so forking may be...

Related: #13, #15, #16

Hmm, I think when I got it working on one of my sites, I just let it download a fresh copy with each deploy. That's definitely slower, but the simplicity...

I'm sorry to hear you feel that way, but I'll stop asking for your input. If you change your mind in the future, please let me know.

> $actual = $phpcs->processString( $payload ); // just psuedocode, not sure if a function like that exists out of the box Looks like we might be able to use `PHP_CodeSniffer\Files\DummyFile`...

It's probably still good to ignore _lines_ that the dev has marked as ignored, though, since they often are things that the dev has manually reviewed. After a quick search,...

I missed this earlier, but there's already [#33472-core](https://core.trac.wordpress.org/ticket/33472) :) It might be worth keeping this open for a bit, just to consider whether that's something we should try to help...

Thanks for the review and tips!

> complex WHERE clauses pieced together from multiple conditional pieces That's mostly what I was thinking about, but also the difficultly escaping table/column names. In general, the problem is the...

I think double prepares often result from building a query conditionally, but passing a static list of arguments to `prepare()`. A better practice IMO would be to pass an _array_...