Kyle Gray

Results 180 comments of Kyle Gray

@farwydi Any chance you could provide the schema for this query? There are a few tables that you're joining on which makes it difficult to debug without the structure of...

Here's the [playground link](https://play.sqlc.dev/p/9359639085584325d3811942133ff00081d5eac367bd3fd9b80df2113c8ba136) for the failing test case from #393

I've reproduced this in the latest version [here](https://play.sqlc.dev/p/b6868489dc7150e910a4ae67ac277b08e4edb60104359653f8dbf6c9c71fa979). `?` placeholders work fine; it's just the sqlc.arg() syntax.

The issue is that sqlc isn't taking into account that `"books".*` is quoted. If you remove the quotes, the generated code is correct. https://play.sqlc.dev/p/333efd4103b524488f7dd3de8fcb483788172191790474562f3997f30cc15247 Obviously still a bug but hopefully...

Support for codegen plugins (both WASM and process-based) was merged recently (https://github.com/kyleconroy/sqlc/pull/1684, https://github.com/kyleconroy/sqlc/pull/1578). As we build out documentation and examples for those, I have no plans to add additional plugins...

Tracking upstream https://github.com/pganalyze/libpg_query/issues/127

Reproduced here: https://play.sqlc.dev/p/8942c5c89e06ea0dc35975ba047936bdf5b0b9ba3daf502103c49c57e457ba5b

@cosban The expectation right now is that `ANY` requires the type cast. I know it's not ideal.

> THIS PR IS NOT THOROUGHLY TESTED AND SHOULD BE REVIEWED WITH A HIGH LEVEL OF SCRUTINY. WHILE IT IS A VERY SMALL CHANGE, I HAVE NO CONSIDERED THE IMPACT...

The tests failed which means this change isn't likely to be safe. Can you provide a bit more context on why you needed to do this?