Skip to content

Comments

fix(quickbooks004): correct error message in check_if_models_use_xr_var#123

Merged
joellabes merged 1 commit intodbt-labs:mainfrom
clkao:fix/quickbooks004-error-message
Feb 23, 2026
Merged

fix(quickbooks004): correct error message in check_if_models_use_xr_var#123
joellabes merged 1 commit intodbt-labs:mainfrom
clkao:fix/quickbooks004-error-message

Conversation

@clkao
Copy link
Contributor

@clkao clkao commented Feb 12, 2026

Summary

  • Fixes error message in tasks/quickbooks004/tests/check_if_models_use_xr_var.sql
  • Said "using_department is in" but should say "using_exchange_rate is not in"
  • Both variable name (copy-paste from quickbooks002/003) and error direction were wrong
  • Cosmetic fix only — does not change pass/fail behavior

Test plan

  • Verify error message is correct by reading the file

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

Wrong variable name (using_department) and wrong direction (is in vs is not in).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@joellabes joellabes merged commit bcb4b42 into dbt-labs:main Feb 23, 2026
3 checks passed
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.

2 participants