Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add shell script syntax checker #4242

Open
jhlegarreta opened this issue Oct 1, 2023 · 0 comments
Open

Add shell script syntax checker #4242

jhlegarreta opened this issue Oct 1, 2023 · 0 comments
Labels
type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots

Comments

@jhlegarreta
Copy link
Member

jhlegarreta commented Oct 1, 2023 •
edited
Loading

Description

Add shell script syntax checker.

Expected coding style

The Utilities folder contains numerous shell scripts that help to maintain ITK. Adding a syntax checker tool (e.g. through pre-commit hooks and GHA workflows) would maybe prevent incompatibilities (e.g. scripts that contain incompatible commands across shells), and bugs or issues from leaking

Actual coding style

No shell script syntax checker is in place.

Versions

master.

Additional Information

Related to issues #796 and #4241.
@jhlegarreta jhlegarreta added the type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots label Oct 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jhlegarreta