Skip to content

Commit f0ca6aa

Browse files
ci: praise be linters & enable external sources in ShellCheck by default
Signed-off-by: Georg Lauterbach <44545919+georglauterbach@users.noreply.github.com>
1 parent ce1bfa7 commit f0ca6aa

File tree

5 files changed

+10
-8
lines changed

5 files changed

+10
-8
lines changed

.vscode/settings.json

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
{
22
"shellcheck.customArgs": [
3-
"--rcfile=.shellcheckrc"
4-
]
3+
"--rcfile=.shellcheckrc",
4+
"--external-sources"
5+
],
56
}

src/lang-sh/devcontainer-feature.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,8 @@
3434
"shellcheck.run": "onSave",
3535
// supply custom arguments to make the most use of ShellCheck
3636
"shellcheck.customArgs": [
37-
"--rcfile=/usr/local/share/dev_containers/features/ghcr_io/georglauterbach/lang_bash/shellcheck.conf"
37+
"--rcfile=/usr/local/share/dev_containers/features/ghcr_io/georglauterbach/lang_bash/shellcheck.conf",
38+
"--external-sources"
3839
]
3940
}
4041
}

src/nodejs/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ Install NodeJS
1717
|-----|-----|-----|-----|
1818
| version | Version of NodeJS to install | string | 25.0.0 |
1919
| architecture | Architecture of NodeJS to install | string | x64 |
20-
| uri | Non-default mirror for the `tar.xz`/`tar.gz` archive that contains NodeJS files. You can specify the version directly or use the string `<<VERSION>>` which is replaced by `node.version` | string | https://nodejs.org/dist/v<<VERSION>>/node-v<<VERSION>>-linux-<<ARCHITECTURE>>.tar.xz |
20+
| uri | Non-default mirror for the `tar.xz`/`tar.gz` archive that contains NodeJS files. You can specify the version directly or use the string `{{VERSION}}` which is replaced by `node.version` | string | https://nodejs.org/dist/v{{VERSION}}/node-v{{VERSION}}-linux-{{ARCHITECTURE}}.tar.xz |
2121
| proxy.http.http.address | A URI for an HTTP proxy | string | - |
2222
| proxy.http.https.address | A URI for an HTTPS proxy | string | - |
2323
| proxy.http.no-proxy.address | A list of URIs to not proxy | string | localhost,127.0.0.1 |

src/nodejs/devcontainer-feature.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,9 @@
1717
"default": "x64"
1818
},
1919
"uri": {
20-
"description": "Non-default mirror for the `tar.xz`/`tar.gz` archive that contains NodeJS files. You can specify the version directly or use the string `<<VERSION>>` which is replaced by `node.version`",
20+
"description": "Non-default mirror for the `tar.xz`/`tar.gz` archive that contains NodeJS files. You can specify the version directly or use the string `{{VERSION}}` which is replaced by `node.version`",
2121
"type": "string",
22-
"default": "https://nodejs.org/dist/v<<VERSION>>/node-v<<VERSION>>-linux-<<ARCHITECTURE>>.tar.xz"
22+
"default": "https://nodejs.org/dist/v{{VERSION}}/node-v{{VERSION}}-linux-{{ARCHITECTURE}}.tar.xz"
2323
},
2424
"proxy.http.http.address": {
2525
"description": "A URI for an HTTP proxy",

src/nodejs/install.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,8 @@ parse_dev_container_options() {
5757

5858
install_node() {
5959
DOWNLOAD_URL=$(printf '%s' "${URI}" | sed \
60-
-e "s|<<VERSION>>|${VERSION}|g" \
61-
-e "s|<<ARCHITECTURE>>|${ARCHITECTURE}|g")
60+
-e "s|{{VERSION}}|${VERSION}|g" \
61+
-e "s|{{ARCHITECTURE}}|${ARCHITECTURE}|g")
6262
FILE_NAME=$(basename "${DOWNLOAD_URL}")
6363
readonly DOWNLOAD_URL FILE_NAME
6464

0 commit comments

Comments
 (0)