Andrew Benton

Re-opening since this is a real issue that I'd like to see closed with https://github.com/sqlc-dev/sqlc/issues/2902.

> If understanding the DO block is out of band, would a structured comment be more achievable ? We have a few "structured comment" proposals in the works. Are you...

> Some change in the struct of the configuration seems to have broken this MR. I will rebase this work on the new configuration logic. Yes, sorry about that. We...

Before getting into code comments, I think @kyleconroy should weigh in on whether this is a good idea and if so whether we should just make this naming behavior the...

I think if we implement #2800 that would give you a way to solve this.

> With Bzlmod, we have a [registry of default `build_directives`](https://github.com/bazelbuild/bazel-gazelle/blob/d2032781c7d4611ce778360ca345d86a97e06956/internal/bzlmod/default_gazelle_overrides.bzl) which would free users from having to specify them manually. For those using Bzlmod and struggling to resolve this manually...

Same thing happens when the package name contains a space. Seems like it would be worth replacing spaces with dashes automatically to create a valid name for the manifest.

Thanks for not losing track of this. From https://www.postgresql.org/docs/current/datatype-datetime.html: "timestamptz is accepted as an abbreviation for timestamp with time zone; this is a PostgreSQL extension." With the new database analyzer...

Yes the issue is not with sqlc's analysis step. I was just trying out the new analyzer to confirm. The new analyzer will have bugs for sure. The error in...

This is still an issue in v1.18.0: https://play.sqlc.dev/p/3931aa6d071068031e5eb03f4e40bd474d22afb2e47aabe0c34e68101e76d2f1. There is already an issue open for adding CAST support: https://github.com/kyleconroy/sqlc/issues/687