Fix bug in line-width control in DAG pretty printer

dbt-labs / metricflow

MetricFlow allows you to define, build, and maintain metrics in code.

https://docs.getdbt.com/docs/build/about-metricflow

Other

1.14k stars 94 forks source link

Fix bug in line-width control in DAG pretty printer #1318

Closed plypaul closed 3 months ago

plypaul commented 3 months ago

This PR fixes a bug with how the maximum width for pretty printing was not correctly computed. Note that SqlStringExpression(node_id=str_0 sql_expr='foo') is longer than expected - this is due to a __repr__ issue that will be addressed in a follow up.