#Go Toolchain
static mountain
I don't think there's any 1.24.0 feature being used. This was bumped from 1.23 to 1.24 in all SDKs right after v0.16.2. I think the "go 1.122" instruction is meant for compatibility while the toolchain one is for keeping up with the times.@oblique geyser, can you give more context?
oblique geyser
🤔 so the toolchain parameter here sets the version of the toolchain to download
if you don't have that version installed
i apparently added this behavior to download it in https://github.com/dagger/dagger/pull/6652 (which matches the default go experience, not the go-in-docker container experience)
but with https://github.com/dagger/dagger/issues/6706, i added some funky error cases
i wonder if we should just delete this 🤔 we already just refuse to run modules that have a higher-than-known go version (because of this hell: https://github.com/dagger/dagger/issues/6706)
will keep thinking about that, but we can downgrade the toolchain for sure for python
remote wigeon
Thank would be great! If you have a patch/PR I'll gladly test it
oblique geyser
remote wigeon
I'm finishing work for few hours, but I'll test this in the evening (in 5h)
oblique geyser
no rush!