| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Projects that author MODEL/AUDIT/METRIC headers in warehouse dialect lose dialect-specific values (column types, audits, macros) when format forces dialect=None. Add an opt-in format.transpile_meta flag to restore the old header rendering while keeping dialect-agnostic headers as the default. Signed-off-by: Alberto Suman <alberto.suman@1komma5grad.com> Co-authored-by: Cursor <cursoragent@cursor.com>
|
@albertosuman-1k5 Based on this I don't see who format.transpile_meta: true would be useful for? On the other side - it is a pain now when the formatter wants to change our columns to something that does not exist - hence the proposed PR #5926 |
Sorry, something went wrong.
|
@albertosuman-1k5 thank you very much for proposing this. I do worry that this all-or-nothing approach will continue to cause people headaches. I believe #5926 by @fresioAS (with a few additions) might get us a quick-fix and set a foundation for a more scaleable solution. |
Sorry, something went wrong.
|
@mday-io We got headaches because of the sudden change in how headers are parsed, change that was not communicated and without an opt-out flag 😅 Why would an opt-out feature that can be removed once everyone had the chance to prepare their projects (let's say in 6 months) would give everyone headers? We're dealing with 2000+ models, roughly half of them change when using the agnostic formatter. At the moment we don't have the capacity to tackle this but we'd like to upgrade to make use of a feature I contributed to |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Summary
Context
#5864 stopped transpiling meta expressions to avoid corrupting SQLMesh DDL booleans. That also rewrote dialect-specific header values (for example DATETIME2(6) → TIMESTAMP(6)). A surgical per-property approach (see #5926 for columns) would need similar handling for audits, labels, macros, and other header properties; this flag restores the previous whole-header behavior as an explicit opt-in instead.
Trade-off: enabling the flag reopens the T-SQL boolean rewrite from #5773 for SQLMesh properties in the header. Documented in docs/reference/configuration.md.
Test plan
Made with Cursor