Subham-KRLX opened a new pull request, #2171:
URL: https://github.com/apache/sedona/pull/2171

   Yes I have read the [Contributor 
Rules](https://sedona.apache.org/latest/community/rule/) and [Contributor 
Development Guide](https://sedona.apache.org/latest/community/develop/)
   
   Yes and the PR name follows the format [GH-1995] Add pre-commit hook to 
auto-insert Apache license headers in shell scripts. Closes #1995
   
   This PR introduces a pre-commit hook configuration to automatically insert 
the Apache License header in all shell script files (*.sh). It includes:
   
   Addition of the shell license insertion hook in .pre-commit-config.yaml 
utilizing the existing Lucas-C/pre-commit-hooks insert-license hook.
   
   Removal of inconsistent license headers and TODO license comments from 
current shell scripts.
   Adjustment to insert the license header immediately after the shebang line 
to maintain script correctness.
   Verification that the license header does not duplicate on multiple runs.
   Updates to various shell scripts to comply with the licensing standard.
   Locally ran pre-commit run insert-license --all-files ensuring no duplicate 
headers and correct placement.
   Manual verification of header insertion after shebang in affected shell 
script files.
   Verified all other pre-commit hooks pass successfully.
   Committed and pushed the changes to the feature branch allowing GitHub 
Actions to validate build and tests.
   
   No this PR does not affect any public API so no need to change the 
documentation.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to