Skip to content

Get all shell scripts passing shellcheck #10795

Open
@Ericson2314

Description

Since #10787, we are now running shellcheck on scripts.

Just like with C/C++ formatting, we are excluding almost all existing files by default. However, the reason for doing that is different than for formatting --- fixing shellcheck lint errors does not have the downside of causing merge conflicts churn, but does require manual edits. Thus we're skipping existing files not because it would be potentially bad to get them passing, but because it is more work than is appropriate for a single PR.

Scripts can one-by-one be made passing, which is a great issue to share between new contributors!

Metadata

Assignees

No one assigned

    Labels

    contributor-experienceDeveloper experience for Nix contributorsgood first issueQuick win for first-time contributorsidea approvedThe given proposal has been discussed and approved by the Nix team. An implementation is welcome.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions