Skip to content

Chore: warn instead of error if description present for non-existent col#5319

Closed
treysp wants to merge 1 commit intomainfrom
trey/col-desc-warning
Closed

Chore: warn instead of error if description present for non-existent col#5319
treysp wants to merge 1 commit intomainfrom
trey/col-desc-warning

Conversation

@treysp
Copy link
Contributor

@treysp treysp commented Sep 8, 2025

For dbt compatibility

@treysp treysp requested a review from crericha September 8, 2025 19:53
@treysp treysp changed the title Chore: warn instead of error when col description provided for non-existent col Chore: warn instead of error if description present for non-existent col Sep 8, 2025
@treysp treysp force-pushed the trey/col-desc-warning branch from 2cdb9f2 to ff32d86 Compare September 8, 2025 20:53
@treysp treysp force-pushed the trey/col-desc-warning branch from ff32d86 to b8170e6 Compare September 8, 2025 20:53
@georgesittas
Copy link
Contributor

georgesittas commented Sep 11, 2025

@treysp this is embarrassing... I forgot that I approved this and came across the same issue, which I solved in almost the exact same way here. Apologies... 🤦

@treysp treysp closed this Sep 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants