Closed ChenyuLInx closed 1 year ago
LG 1/30/2023 @ChenyuLInx and @stu-k are going to pair and better refine this ticket for grooming next time.
I pushed the changes I think fix the change to manifest generation abstraction to this branch. Any work around this should probably be relegated to testing!
After more code diving we discovered that we override init so the change in function sig isn't an issue.
As the exploration we did with #113, it is possible to continue supporting
dbt-core
for one more version withdbt-rpc
. As the API-ification project moving on, there are likely new things that we need to adjust to makedbt-rpc
working(i.e. manifest got refactored). This work should be estimated and being worked on after we mergefeature/click-cli
indbt-core
tomain
.We should make sure all integration tests in
dbt-rpc
pass with latestmain
branch ofdbt-core