apache / apache/teaclave

Add shell script style

Open
#612 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
No language data
Stars
805
Forks
155
PR merge metrics
No merged PRs in 30d

Description

We should have a style guide for bash scripts.

As discussed in https://github.com/apache/incubator-teaclave/pull/609, environment variables and internal shell variables are capitalized. All other variable names should be lower case. Some shell scripts in the project don't follow this convection.

Here is an example:
https://google.github.io/styleguide/shellguide.html

In addition, we can add a shell scripts format checker to the build pipeline.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.