flox / flox

Developer environments you can take with you
https://flox.dev
GNU General Public License v2.0
2.12k stars 50 forks source link

Comment out dependency fetching in auto-setup activation scripts #1382

Open jennymahmoudi opened 2 weeks ago

jennymahmoudi commented 2 weeks ago

In a user research session, the participant accepted the suggested manifest changes after Flox detected a Go project. In this case, the user had to turn off the VPN and so when go tried to get the private dependencies, Flox wasn't able to activate the shell.

We might want to remove the line to get dependencies since it's not required for the go setup, and might cause unexpected issues like this.

image

ysndr commented 2 weeks ago

This issue should be more general then. At least the python hook is also doing a pip install (or equivalent).

mkenigs commented 2 weeks ago

For now let's comment out go get, pip install, npm i etc. We'll change if @jennymahmoudi changes her mind