Switch the pre-commit hook used to run `shfmt`

[mcdonnnj]

🗣 Description

This pull request switches the pre-commit hook used to run shfmt from cisagov/pre-commit-shfmt to scop/pre-commit-shfmt. It also updates the arguments to shfmt to use the long options that are now available.

💭 Motivation and context

The new hook actually bundles the binary releases from shfmt into a Python package that is used for the hook. This removes the need to have the shfmt tool installed before you are able to use the hook.

🧪 Testing

Automated tests pass.

✅ Pre-approval checklist

• [x] This PR has an informative and human-readable title.
• [x] Changes are limited to a single goal - eschew scope creep!
• [x] All relevant type-of-change labels have been added.
• [x] I have read the CONTRIBUTING document.
• [x] These code changes follow cisagov code standards.
• [x] Tests have been added and/or modified to cover the changes in this PR.
• [x] All new and existing tests pass.

✅ Post-merge checklist

• [ ] Archive the cisagov/pre-commit-shfmt repository.

[mcdonnnj]

Excellent! 👍

Does this mean that we can now make https://github.com/cisagov/pre-commit-shfmt a public archive?

Yes that's the plan once this is merged.

[dav3r]

Excellent! 👍 Does this mean that we can now make https://github.com/cisagov/pre-commit-shfmt a public archive?

Yes that's the plan once this is merged.

Great! You should add that as a post-merge checklist item so that we don't forget about it.