-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Run dbt on WebAssembly #6146
Run dbt on WebAssembly #6146
Conversation
unit tests pass, these integration tests fail for me locally (not sure if my setup is bad): full logs are here: https://gist.github.com/arieldbt/7a5d742ecfe64e75f17bbb9d898e33b1
|
Annnnnd I got the integration tests to pass by running this command instead (updated the full log): https://gist.github.com/arieldbt/7a5d742ecfe64e75f17bbb9d898e33b1
|
@arieldbt Thanks for reopening! Most of these changes look reasonable. I'm excited that they actually leave us with cleaner / more pluggable interfaces for things like external requests. And the new click CLI → top-level APIs should make it even easier to invoke Considerations:
Ultimately, I think the folks on the Execution team are best situated to review this. |
Going to close this one for now. It remains a very cool idea—but not something we can make immediate progress on, nor do we have a tangible use case that we can invest in supporting on an ongoing basis. |
Description
Fixes for the linter on this PR: #5803
Checklist
changie new
to create a changelog entry