Add new context argument to _get_responses method

dbt-labs / dbt-rpc

A server that can compile and run queries in the context of a dbt project. Additionally, it provides methods that can be used to list and terminate running processes.

https://docs.getdbt.com/reference/commands/rpc

Apache License 2.0

19 stars 7 forks source link

Add new context argument to _get_responses method #117

Closed anubhavsharma515 closed 1 year ago

anubhavsharma515 commented 1 year ago

Fixes #116.

From the linked issue, a recent modification to the json-rpc parent JSONRPCResponseManager https://github.com/pavlov99/json-rpc/pull/122/files#diff-71ff9d558b2e51825f517c97faa2f119ab0aaf26aaff43e36731719017cce06eR75 class adds a new argument that is causing an error in the overridden version in the custom dbt-rpc ResponseManager class.

This PR essentially just adds in the same context argument to the _get_responses method, which is being called from the parent handle_request method.

cla-bot[bot] commented 1 year ago

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, don't hesitate to ping @drewbanin.

CLA has not been signed by users: @anubhavsharma515