Skip to content
This repository has been archived by the owner on Dec 4, 2024. It is now read-only.

v1 compatibility changes #75

Merged
merged 2 commits into from
Dec 7, 2021
Merged

v1 compatibility changes #75

merged 2 commits into from
Dec 7, 2021

Conversation

joellabes
Copy link
Contributor

Description & motivation

  • Change require-dbt-version to [">=1.0.0", "<2.0.0"]
  • Bump dbt-utils dependency
  • Replace source-paths and data-paths with model-paths and seed-paths respectively
  • Rename data and analysis directories to seeds and analyses respectively
  • Replace dbt_modules with dbt_packages in clean-targets
  • Put --warn-error back into CI jobs
  • Expand pip install dbt command to cover all adapters

Checklist

  • I have verified that these changes work locally
  • I have updated the README.md (if applicable)
  • I have added tests & descriptions to my models (and macros if applicable)

@joellabes joellabes requested a review from jasnonaz December 4, 2021 04:09
jasnonaz
jasnonaz approved these changes Dec 7, 2021
View reviewed changes
Copy link

@jasnonaz jasnonaz left a comment

Choose a reason for hiding this comment

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

Let this release ring from sea to shining analyses

@joellabes joellabes merged commit 65d4374 into master Dec 7, 2021
@joellabes joellabes deleted the full-v1-compatibility branch December 7, 2021 21:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jasnonaz jasnonaz jasnonaz approved these changes

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
@joellabes @jasnonaz