mirror of
https://github.com/koalaman/shellcheck.git
synced 2025-10-03 19:29:44 +08:00
Created SC3033 (markdown)
33
SC3033.md
Normal file
33
SC3033.md
Normal file
@@ -0,0 +1,33 @@
|
||||
## In POSIX sh, naming functions outside [a-zA-Z_][a-zA-Z0-9_]* is undefined.
|
||||
|
||||
### Problematic code:
|
||||
|
||||
```sh
|
||||
#!/bin/sh
|
||||
run-tests() {
|
||||
make test
|
||||
}
|
||||
```
|
||||
|
||||
### Correct code:
|
||||
|
||||
```sh
|
||||
#!/bin/sh
|
||||
run_tests() {
|
||||
make test
|
||||
}
|
||||
```
|
||||
|
||||
### Rationale:
|
||||
|
||||
Your script uses a shell feature not supported by the shebang. Either rewrite the script to be portable, or change the shebang to explicitly require a shell like Bash.
|
||||
|
||||
In this case, you can rename your functions to follow the same naming requirements as variables, as these are the only names guaranteed to be valid function names in POSIX.
|
||||
|
||||
### Exceptions:
|
||||
|
||||
None
|
||||
|
||||
### Related resources:
|
||||
|
||||
* Help by adding links to BashFAQ, StackOverflow, man pages, POSIX, etc!
|
Reference in New Issue
Block a user