Skip to content

Add manual hook chmod to set file permissions for md files #418

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
jbampton wants to merge 1 commit into
base: trunk
Choose a base branch
from
Open

Add manual hook chmod to set file permissions for md files #418

jbampton wants to merge 1 commit into from

Conversation

@jbampton
Copy link
Member

@jbampton jbampton commented Sep 6, 2025

chmod is not available on standard Windows so set this hook to manual.

Linux and macOS users can run this hook locally for a full repository clean up.

Also added the manual stage hook to GitHub Actions so we don't lose coverage

Example usage:

https://github.com/Lucas-C/pre-commit-hooks?tab=readme-ov-file#usage

https://github.com/apache/sedona/blob/3a1e9b7515b4dea994969f7f3c9ff43cf700e177/.pre-commit-config.yaml#L81

Finally renamed the workflow linter.yml to pre-commit.yml
@jbampton
Add manual hook chmod to set file permissions for md files
4cbceec 
chmod is not available on standard Windows so set this hook to manual.

Linux and macOS users can run this hook locally for a full repository clean up.

Also added the manual stage hook to GitHub Actions so we don't lose coverage

Example usage:

https://github.com/Lucas-C/pre-commit-hooks?tab=readme-ov-file#usage

https://github.com/apache/sedona/blob/3a1e9b7515b4dea994969f7f3c9ff43cf700e177/.pre-commit-config.yaml#L81

Finally renamed the workflow `linter.yml` to `pre-commit.yml`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@jbampton