Skip to content

feat(constraints) : adding constraints and descriptive documentation in demo models #86

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat(constraints) : adding constraints and descriptive documentation in demo models #86

wants to merge 2 commits into from

Conversation

Fujistone
Copy link
Collaborator

@Fujistone Fujistone commented Mar 4, 2025 •
edited
Loading

Few points :

  • do we keep the target schema as such or does it need to be defined ?
  • redundancy of keeping data tests AND constraints with the same characteristics on the same columns ?
  • is it necessary to define constraints at both model AND columns level ?

@JeanLouisLamezec JeanLouisLamezec force-pushed the add_constraints branch 2 times, most recently from 80fe871 to 3514347 Compare March 17, 2025 14:24
Fujistone and others added 2 commits March 17, 2025 16:25
@Fujistone @JeanLouisLamezec
feat(constraints) : adding constraints and descriptive documentation …
9dcf531 
…to columns in demo models

fix(matomo): rm campaign_name frm matomo_campaign_source_medium

fix(matomo_campaign): rm matomo_content

wip

feat(meetings + proposals): use new macro to check column.dtype

feat(proposals): add macro to deal with old_state and state

rm club_utilisateurs from ci

feat(ps_belge) : adding Ps Belge dbt project

feat(constraints) : adding constraints and descriptive documentation to columns in demo models

feat(constraints) : removing model level constraints and adding constraints packages in demo

wip

feat(constraints) : removing model level constraints and adding constraints packages in demo
@JeanLouisLamezec
feat(components): test rm properties
e3f22a3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Fujistone @JeanLouisLamezec