-
Notifications
You must be signed in to change notification settings - Fork 207
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
chore: update devcontainer to use proper docker compose #2162
chore: update devcontainer to use proper docker compose #2162
Conversation
Signed-off-by: Bryce McMath <bryce.j.mcmath@gmail.com>
|
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.
Thanks! What's broken when building locally?
@TimoGlastra
|
pnpm: 9.15.3 |
Interesting. I've been on node 20/22 for a while. Could you try using the latest of one of these two? |
@TimoGlastra tried node 20, got less errors, but still broken with the same kind of message:
Tried with node 22, back to the multiple errors, looks roughly the same as the error I get on 18:
Just a shot in the dark, what Python version are you using and what did you use to install it? I'm wondering if the build step is failing cause of some bindings issue. |
…oundation#2162) Signed-off-by: Bryce McMath <bryce.j.mcmath@gmail.com> Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
This is just an addition for devs that want to run the full test suite and need the docker compose setup, but have issues building locally (such as myself lol 🥲)