Skip to content

Comments

Change interpreter from bash to sh to support more shells#336

Merged
mstruebing merged 2 commits intoeditorconfig-checker:masterfrom
mzk:os-support-alpine-docker
Nov 21, 2025
Merged

Change interpreter from bash to sh to support more shells#336
mstruebing merged 2 commits intoeditorconfig-checker:masterfrom
mzk:os-support-alpine-docker

Conversation

@mzk
Copy link
Contributor

@mzk mzk commented Nov 21, 2025 •
edited
Loading

add Docker alpine support.

> vendor/bin/ec .
env: can't execute 'bash': No such file or directory
Script vendor/bin/ec . handling the ec event returned with error code 127
Cleaning up project directory and file based variables

mstruebing
mstruebing approved these changes Nov 21, 2025
View reviewed changes
Copy link
Member

@mstruebing mstruebing left a comment

Choose a reason for hiding this comment

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

LGTM!

I was suprised that the function keywords needs to be deleted for sh and I've even tried it.

Did a little bit of research and found this: https://www.shellcheck.net/wiki/SC2112

@mstruebing mstruebing merged commit 7b20ee1 into editorconfig-checker:master Nov 21, 2025
8 checks passed
@mzk mzk deleted the os-support-alpine-docker branch November 24, 2025 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mstruebing mstruebing mstruebing approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mzk @mstruebing