Fix case-sensitivity in validation warning

courtneyholcomb commented 1 month ago

Problem

In DSI 0.7.2, we released a validation warning that was triggered if a user added a lowercase granularity to a filter in their YAML. This should not have been case sensitive.

Solution

DSI 0.7.3 fixes that issue, so this PR bumps to that version of DSI. See the commit that fixes it here: https://github.com/dbt-labs/dbt-semantic-interfaces/pull/352

Checklist

[ ] I have read the contributing guide and understand what's expected of me.
[ ] I have run this code in development, and it appears to resolve the stated issue.
[ ] This PR includes tests, or tests are not required or relevant for this PR.
[ ] This PR has no interface changes (e.g., macros, CLI, logs, JSON artifacts, config files, adapter interface, etc.) or this PR has already received feedback and approval from Product or DX.
[ ] This PR includes type annotations for new and modified functions.

github-actions[bot] commented 1 month ago

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

github-actions[bot] commented 1 month ago

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.17%. Comparing base (396cf2d) to head (ebf278f). Report is 2 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #10807 +/- ## ========================================== - Coverage 89.21% 89.17% -0.05% ========================================== Files 183 183 Lines 23457 23457 ========================================== - Hits 20928 20917 -11 - Misses 2529 2540 +11 ``` | [Flag](https://app.codecov.io/gh/dbt-labs/dbt-core/pull/10807/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dbt-labs) | Coverage Δ | | |---|---|---| | [integration](https://app.codecov.io/gh/dbt-labs/dbt-core/pull/10807/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dbt-labs) | `86.41% <ø> (-0.13%)` | :arrow_down: | | [unit](https://app.codecov.io/gh/dbt-labs/dbt-core/pull/10807/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dbt-labs) | `62.09% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dbt-labs#carryforward-flags-in-the-pull-request-comment) to find out more. | [Components](https://app.codecov.io/gh/dbt-labs/dbt-core/pull/10807/components?src=pr&el=components&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dbt-labs) | Coverage Δ | | |---|---|---| | [Unit Tests](https://app.codecov.io/gh/dbt-labs/dbt-core/pull/10807/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dbt-labs) | `62.09% <ø> (ø)` | | | [Integration Tests](https://app.codecov.io/gh/dbt-labs/dbt-core/pull/10807/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dbt-labs) | `86.41% <ø> (-0.13%)` | :arrow_down: |

dbt-labs / dbt-core