-
Notifications
You must be signed in to change notification settings - Fork 40
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.
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
🐛 Changed dbt packages in requirements.txt #855
🐛 Changed dbt packages in requirements.txt #855
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added comments
CHANGELOG.md
Outdated
@@ -8,9 +8,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 | |||
### Added | |||
|
|||
### Fixed | |||
- Changed requirements.txt to level up version of dbt-sqlserver in order to fix bug with `MAXRECURSION` error in dbt_run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Changed requirements.txt to level up version of dbt-sqlserver in order to fix bug with `MAXRECURSION` error in dbt_run | |
- Changed `requirements.tx`t to level up version of `dbt-sqlserver` in order to fix bug with `MAXRECURSION` error in dbt_run. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
CHANGELOG.md
Outdated
|
||
### Changed | ||
|
||
### Removed | ||
- Removed dbt-core==1.3.2 from requirements.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Removed dbt-core==1.3.2 from requirements.txt | |
- Removed `dbt-core==1.3.2` from `requirements.txt`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Summary
The PR containing changes in requirements.txt file. Dbt-sqlserver were upgraded to djagoda's fork in order to fix bug in dbt.
Importance
It is important because without it some of the Transform&Catalog flows will be failing on Prefect due to bug in current version of dbt-sqlserver. Linked issue: #854
Checklist
This PR:
CONTRIBUTING.md
CHANGELOG.md