You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
in the past we had latest tag for both, compiler & node. this recently lead us to a problem with SDK incompatibility to latest compiler version v7.0.1
I think it makes sense to fetch latest (major) release on project initialization. if e.g. aeproject would define 6 for node and 6 for the compiler it should replace the tags accordingly in docker-compose.yml (which we recently introduced here: #432)
aeproject would then use the version defined in docker-compose.yaml by default to start the environment. only if options nodeVersion or compilerVersion is provided a custom version should be used.
this way we would ensure that a user doesn't run into unforeseen problems.
also I see one small issue with the other approach. in case of new releases docker wouldn't automatically download the new node version if not explicitly forced by the user.
ok, that's good to know. anyway, I doubt we will see tags like 6 or 7 on dockerhub based on recent discussion.
using the proposed approach here we would definitely avoid unexpected results based on implicit infrastructure changes. we'd always select the version when the project is initialized and then leave it up to the devs to update node/compiler. I actually like that
in the past we had
latest
tag for both, compiler & node. this recently lead us to a problem with SDK incompatibility to latest compiler versionv7.0.1
I think it makes sense to fetch latest (major) release on project initialization. if e.g. aeproject would define
6
for node and6
for the compiler it should replace the tags accordingly in docker-compose.yml (which we recently introduced here: #432)aeproject would then use the version defined in
docker-compose.yaml
by default to start the environment. only if optionsnodeVersion
orcompilerVersion
is provided a custom version should be used.this way we would ensure that a user doesn't run into unforeseen problems.
@thepiwo what do you think?
The text was updated successfully, but these errors were encountered: