Skip to content

Fix: (typo) desctiption -> description in base_ga4__pseudonymous_users.yml - #397

Merged
mtcarlone merged 4 commits into
Velir:mainfrom
masukai:fix/typo-desctiption-pseudonymous-users
Sep 8, 2026
Merged

mtcarlone merged 4 commits into
Velir:mainfrom
masukai:fix/typo-desctiption-pseudonymous-users

Conversation

@masukai

@masukai masukai commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Description & motivation

Fixes a typo in models/staging/base/base_ga4__pseudonymous_users.yml: the last_updated_date column uses desctiption: instead of description:.

This isn't purely cosmetic — because desctiption is an unrecognized key:

  • The intended description is silently dropped, so last_updated_date ends up undocumented in the generated catalog/docs.
  • dbt 1.10+ emits a CustomKeyInObjectDeprecation warning on every parse, which surfaces in every downstream project that installs this package.

Same class of fix as #385 (Clint → Client).

Checklist

  • I have verified that these changes work locally
  • I have updated the README.md (if applicable) — N/A
  • I have added tests & descriptions to my models (and macros if applicable) — restores the dropped description
  • I have run dbt test and python -m pytest . to validate existing tests — N/A, single YAML key rename with no SQL/logic change

Single-character key rename (desctiption → description); no model SQL or logic touched.

…s.yml

The `desctiption` key on the `last_updated_date` column is unrecognized by dbt,
so the intended description is silently dropped (the column ends up undocumented)
and dbt 1.10+ raises a CustomKeyInObjectDeprecation warning on every parse for
downstream projects that install this package.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dgitis

dgitis commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Thank you for submitting this.

Github seems to be having a moment. I can't review the PR but I can request that I review the PR.

Maybe @mtcarlone will have more luck.

I'm not in the Velir organization but he is.

@masukai

masukai commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Thanks @dgitis — really appreciate the quick look and the hand-off to @mtcarlone! No rush on my end. Happy to tweak anything if needed once GitHub settles down. 🙏

@mtcarlone mtcarlone left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@mtcarlone

Copy link
Copy Markdown
Collaborator

This will go out September 8, end of day.

@mtcarlone
mtcarlone merged commit 2a45f04 into Velir:main Sep 8, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants