Skip to content

chore: auto-format shell#6780

Merged
holmanb merged 7 commits intomainfrom
holmanb/shfmt
Mar 25, 2026
Merged

chore: auto-format shell#6780
holmanb merged 7 commits intomainfrom
holmanb/shfmt

Conversation

@holmanb
Copy link
Copy Markdown
Member

@holmanb holmanb commented Mar 4, 2026

Proposed Commit Message

chore: auto-format shell

Make `tox` -e do_format` and `tox -e check_format` enforce
formatting of json and shell scripts.

Remove unmaintained shell scripts.

Update GH Actions.

Additional Context

Test Steps

Merge type

  • Squash merge using "Proposed Commit Message"
  • Rebase and merge unique commits. Requires commit messages per-commit each referencing the pull request number (#<PR_NUM>)

Make `tox` -e do_format` and `tox -e check_format` enforce
formatting of json and shell scripts.

Remove unmaintained shell scripts.

Update GH Actions.

```
shfmt --list --case-indent --indent=4 --space-redirects -w
```

Using v3.12.0
@holmanb holmanb assigned holmanb and unassigned holmanb Mar 16, 2026
@blackboxsw blackboxsw self-assigned this Mar 25, 2026
Copy link
Copy Markdown
Collaborator

@blackboxsw blackboxsw left a comment

Choose a reason for hiding this comment

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

This is the same content previously approved and with conflict resolved. +1

@holmanb holmanb merged commit a5f6267 into main Mar 25, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants