Skip to content

refactor(gooddata-dbt): use CatalogGeoAreaConfig type for geo_area_config#1386

Merged
hkad98 merged 1 commit intogooddata:masterfrom
hkad98:jkd/geo
Mar 4, 2026
Merged

refactor(gooddata-dbt): use CatalogGeoAreaConfig type for geo_area_config#1386
hkad98 merged 1 commit intogooddata:masterfrom
hkad98:jkd/geo

Conversation

@hkad98
Copy link
Contributor

@hkad98 hkad98 commented Mar 4, 2026

Summary

  • Replace raw dict[str, dict[str, str]] with typed CatalogGeoAreaConfig from gooddata-sdk for the geo_area_config field in DbtModelMetaGoodDataColumnProps
  • Enables cattrs validation at dbt manifest parse time, so malformed or misnamed config keys (e.g. camelCase geoAreaConfig) fail early with a clear error instead of silently producing None

…nfig field

Replace raw dict[str, dict[str, str]] with the typed CatalogGeoAreaConfig
from gooddata-sdk. This enables cattrs validation at dbt manifest parse
time, so malformed or misnamed config keys (e.g. camelCase geoAreaConfig)
fail early with a clear error instead of silently producing None.

jira: TRIVIAL
risk: nonprod
@hkad98 hkad98 requested review from jaceksan, lupko and pcerny as code owners March 4, 2026 13:17
@hkad98 hkad98 enabled auto-merge March 4, 2026 13:18
@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.67%. Comparing base (c00ca3b) to head (376dde4).
⚠️ Report is 39 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1386       +/-   ##
===========================================
+ Coverage    0.00%   77.67%   +77.67%     
===========================================
  Files         158      224       +66     
  Lines       11048    14425     +3377     
===========================================
+ Hits            0    11205    +11205     
+ Misses      11048     3220     -7828     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hkad98 hkad98 merged commit 93d0c4d into gooddata:master Mar 4, 2026
11 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