Quigley Malcolm
Quigley Malcolm
We should probably move this issue to dbt-semantic-interfaces
> Should we create a follow up ticket to update our code to make it compatible with protobuf 5.0? Assuming people might install dbt-core with other things which might require...
I've added the `artifact-minor-upgrade` label because the files in `artifacts` have been changed, but it's just the imports.
I think it's debatable if a changie doc is needed for this. We've added a dependency, but it's only a dev dependency. We've made changes in the source code that...
> I'm not sure why the artifacts check is failing in CI - it looks like the action itself is erroring? But LMK if there's something I need to fix...
@jtcohen6, exact same question as on #9446. I noticed on #7440 you mentioned [in a comment](https://github.com/dbt-labs/dbt-core/issues/7440#issuecomment-1525455253) that we intentionally didn't move `meta` under `config` for a subset of node types....
@jtcohen6 I noticed on #7440 you mentioned [in a comment](https://github.com/dbt-labs/dbt-core/issues/7440#issuecomment-1525455253) that we intentionally didn't move `meta` under `config` for a subset of node types. Would you be able to speak...
Additionally I would be a proponent of getting the fix backported to 1.7.latest
Should this be a location name instead of a timezone abbreviation? @Jstein77
`start_time` and `end_time` should be datetime objects on the parsed node. DSI validations will handle checking that the time dimension actually exists. Mashumaro will handle parsing the yaml to the...