Skip to content

fix(shell): resolve ShellCheck SC3043 warnings - #2433

Open
enochval wants to merge 1 commit into
nextcloud:masterfrom
enochval:issue-2428-shellcheck-fix
Open

fix(shell): resolve ShellCheck SC3043 warnings#2433
enochval wants to merge 1 commit into
nextcloud:masterfrom
enochval:issue-2428-shellcheck-fix

Conversation

@enochval

Copy link
Copy Markdown

fix(shell): resolve ShellCheck SC3043 warnings in multiple entrypoint scripts

Dropped 'local' keyword and updated variable names for POSIX compliance. Modified scripts in 29, 30, 31, and main docker-entrypoint.sh.

Fixes: #2428

@brlin-tw brlin-tw left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@joshtrichards
joshtrichards self-requested a review September 23, 2025 03:59
@enochval
enochval force-pushed the issue-2428-shellcheck-fix branch from 18a2280 to 5e193f1 Compare August 2, 2026 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug image

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"In POSIX sh, 'local' is undefined." ShellCheck problem in the entrypoint scripts

4 participants