-
-
Notifications
You must be signed in to change notification settings - Fork 37
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
Support dbt 1.9 #143
Support dbt 1.9 #143
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
3070790
to
a1ae809
Compare
a1ae809
to
364cc97
Compare
I see some issues with the documentation build, I think related to the Happy to take a look later today and try to get this fixed. Everything else looks good to me. |
@millin Hope you don't mind I've pushed a few commits to clear the remaining CI errors. Essentially, seems like something changed in the readthedocs builds and poetry and readthedocs are now installing/expecting dependencies in different virtualenvs. I ended up solving the issue with the poetry export plugin, and outputting a I've also removed the committed Moreover, Snyk was reporting a vulnerability on zipp < 3.19.1. I've pinned zipp >= 3.19.1, but the alert didn't clear. Anyways, with the removal of All that being said, I think CI should build successfully now. I'll take a closer look at this PR tomorrow or on Friday and get it merged+released then. Amazing work! |
I'm glad you were able to correct CI errors 👍 |
I don't see anything to comment here, work looks great! With CI issues resolved, let's ship this. |
No description provided.