Precommit removes shebang from executable

[webb-ben]

Description
Pre-commit action removes the shebangs from an executable shell script when adding the header.

Steps to Reproduce
Steps to reproduce the behavior:

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots/Tracebacks
If applicable, add screenshots to help explain your problem.

Environment

• OS:
• Python version:
• Python env/venv:

Additional context
Add any other context about the problem here.

[C-Loftus]

imo due to issues like these I think we probably shouldn't implement a license precommit hook ourselves since it already exists. There are a lot of edge cases like this that are easy to miss and a lot of different languages to handle.

We also should probably support spdx license identifiers as the standard which is another thing to add.

[webb-ben]

My understanding was https://github.com/google/addlicense is not a precommit action. What are you using?

[webb-ben]

Would also really like to include who owns the file in the header.

[webb-ben]

We will also need the action to work for some internal private license

[C-Loftus]

I am using the precommit action from the fork that was used for this pr. It was approved today so imagine it will be merged. google/addlicense#160

[C-Loftus]

We will also need the action to work for some internal private license

addlicense works with arbitary licenses by specifying a path to a license file just as an fyi