diff options
| author | Danilo M. <danix@danix.xyz> | 2026-09-17 09:09:57 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-09-17 09:09:57 +0200 |
| commit | 04542528f018678b64e0003eab078858c31f196e (patch) | |
| tree | 31ec31c0b3d04f684f9d8f71ffb3fadd01c58b5e | |
| parent | a55051da4a286649361d1fc811ecfccacde17695 (diff) | |
| download | conky-theme-udt-04542528f018678b64e0003eab078858c31f196e.tar.gz conky-theme-udt-04542528f018678b64e0003eab078858c31f196e.zip | |
fix: let the shellcheck directive bind to the source line
`set -a; . "$ENV_FILE"; set +a` on one line meant the
`# shellcheck source=/dev/null` above it attached to the `set`, not the
`.`, so SC1090 fired anyway. Split across lines, the directive binds and
the script lints clean.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
| -rwxr-xr-x | bin/weather-fetch.sh | 4 | ||||
| -rw-r--r-- | docs/superpowers/plans/2026-09-17-weather-widget.md | 4 |
2 files changed, 6 insertions, 2 deletions
diff --git a/bin/weather-fetch.sh b/bin/weather-fetch.sh index 76aa07b..d4e31c0 100755 --- a/bin/weather-fetch.sh +++ b/bin/weather-fetch.sh @@ -17,8 +17,10 @@ if [ ! -r "$ENV_FILE" ]; then exit 1 fi +set -a # shellcheck source=/dev/null -set -a; . "$ENV_FILE"; set +a +. "$ENV_FILE" +set +a if [ -z "${KEY:-}" ]; then echo "weather-fetch: KEY is empty in $ENV_FILE" >&2 diff --git a/docs/superpowers/plans/2026-09-17-weather-widget.md b/docs/superpowers/plans/2026-09-17-weather-widget.md index 36e2dc1..01cd767 100644 --- a/docs/superpowers/plans/2026-09-17-weather-widget.md +++ b/docs/superpowers/plans/2026-09-17-weather-widget.md @@ -762,8 +762,10 @@ if [ ! -r "$ENV_FILE" ]; then exit 1 fi +set -a # shellcheck source=/dev/null -set -a; . "$ENV_FILE"; set +a +. "$ENV_FILE" +set +a if [ -z "${KEY:-}" ]; then echo "weather-fetch: KEY is empty in $ENV_FILE" >&2 |