check-full faalt op main door ShellCheck SC2012 in whisper-bundelscript #2560

New issue

Closed

opened 2026-10-23 18:46:54 +00:00 by brenno · 2 comments

brenno commented

2026-10-23 18:46:54 +00:00

Owner

Reproductie op origin/main (00bef8b42): make check-full stopt bij shellcheck scripts/.sh met SC2012 op scripts/bundle_whisper_cli.sh:30. De regel gebruikt ls -d .../.app | head -1. Het script is ongewijzigd ten opzichte van main op de bestandslijstbranch. Gevolg: de uitgebreide lokale poort kan niet groen worden en een PR kan niet volledig gevalideerd worden.

Acceptatie: kies een .app via een ShellCheck-veilige route zonder de bestaande foutafhandeling of macOS-bundelbestemming te veranderen; shellcheck scripts/*.sh en make check-full zijn groen. Fix en PR afzonderlijk van de bestandslijstwijziging (#2558).

Reproductie op origin/main (00bef8b42): make check-full stopt bij shellcheck scripts/*.sh met SC2012 op scripts/bundle_whisper_cli.sh:30. De regel gebruikt ls -d .../*.app | head -1. Het script is ongewijzigd ten opzichte van main op de bestandslijstbranch. Gevolg: de uitgebreide lokale poort kan niet groen worden en een PR kan niet volledig gevalideerd worden. Acceptatie: kies een .app via een ShellCheck-veilige route zonder de bestaande foutafhandeling of macOS-bundelbestemming te veranderen; shellcheck scripts/*.sh en make check-full zijn groen. Fix en PR afzonderlijk van de bestandslijstwijziging (#2558).

brenno added the

accepted

in-progress

bug

labels

2026-10-23 18:46:54 +00:00

brenno commented

2026-10-23 18:47:07 +00:00

Author

Owner

Ik pak dit afzonderlijk op in codex/fix-shellcheck-baseline. Scope: uitsluitend scripts/bundle_whisper_cli.sh. Daarna draai ik de projectpoort en lever ik een eigen PR, zodat #2558 op een groene main kan worden gevalideerd.

brenno referenced this issue from a commit

2026-10-23 19:19:53 +00:00

Herstel ShellCheck in whisper-bundelscript (#2560)

brenno referenced this issue from a pull request that will close it,

2026-10-23 19:19:58 +00:00

Herstel ShellCheck in whisper-bundelscript (#2560) #2563

brenno closed this issue

2026-10-23 19:21:40 +00:00

brenno referenced this issue from a commit

2026-10-23 19:21:40 +00:00

Merge pull request 'Herstel ShellCheck in whisper-bundelscript (#2560)' (#2563) from codex/fix-shellcheck-baseline into main

brenno commented

2026-10-23 22:24:42 +00:00

Author

Owner

Opgelost en gemerged via PR #2563 op main (commit 26448faca). ShellCheck en make check-full waren groen op die commit. De macOS-CI van deze merge werd door latere main-pushes geannuleerd; de opvolgende run op actuele main wordt gevolgd.

Opgelost en gemerged via PR #2563 op main (commit 26448faca). ShellCheck en make check-full waren groen op die commit. De macOS-CI van deze merge werd door latere main-pushes geannuleerd; de opvolgende run op actuele main wordt gevolgd.

brenno removed the

in-progress

label

2026-10-23 22:24:47 +00:00