-
Notifications
You must be signed in to change notification settings - Fork 50
Fix: Zig version update #158
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
base: main
Are you sure you want to change the base?
Conversation
| "proposals": [ | ||
| "latest", | ||
| "0.10.0" | ||
| "lts", |
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.
| "lts", | |
| "latest", | |
| "stable", |
| "0.12.1" | ||
| ], | ||
| "default": "latest", | ||
| "default": "lts", |
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.
| "default": "lts", | |
| "default": "stable", |
| then | ||
| # for latest we download the latest *release* version | ||
| DOWNLOAD_URL=$(curl -sSL $INDEX_URL | jq -r 'to_entries[0].value."'"$ARCH"'-linux".tarball') | ||
| elif [[ "$ZIG_VERSION" == "lts" ]]; then |
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.
| elif [[ "$ZIG_VERSION" == "lts" ]]; then | |
| elif [[ "$ZIG_VERSION" == "stable" ]]; then |
| | version | Select the Zig version you would like to install | string | latest | | ||
| | latest | This is latest version of zig which is still not released yet. Under development for developers use. You can use it as Beta release and post any issues on zig github repo.| | ||
| | lts | This is the Last released version of zig which is more stable and efficient.| |
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.
We are keeping a single option here, you can reuse its description from src/zig/devcontainer-feature.json. Following example change from this comment:
| | version | Select the Zig version you would like to install | string | latest | | |
| | latest | This is latest version of zig which is still not released yet. Under development for developers use. You can use it as Beta release and post any issues on zig github repo.| | |
| | lts | This is the Last released version of zig which is more stable and efficient.| | |
| | version | Select the Zig version you would like to install. `stable` installs the latest stable release, while `latest` installs the most recent version (which may be a development release | string | stable | |
| ], | ||
| "default": "latest", | ||
| "default": "lts", | ||
| "description": "Select the Zig version you would like to install" |
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.
We provide more context here, for example:
| "description": "Select the Zig version you would like to install" | |
| "description": "Select the Zig version you would like to install. `stable` installs the latest stable release, while `latest` installs the most recent version (which may be a development release)" |
koralowiec
left a comment
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.
Heyo!
I left few change requests. Generally, I don't see any info about Zig having LTS versions, so using lts may be misleading. I would propose to use stable instead
| { | ||
| "name": "Zig (via ziglang.org)", | ||
| "id": "zig", | ||
| "version": "1.1.2", |
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.
Version bump will be required
| "version": "1.1.3", |
update README and JSON options for Zig version selection