Skip to content
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.

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

Creating a pre-commit hook #53

Open
tueda opened this issue Jan 20, 2025 · 1 comment
Open

Creating a pre-commit hook #53

tueda opened this issue Jan 20, 2025 · 1 comment
Assignees

Comments

@tueda
Copy link

tueda commented Jan 20, 2025

Thanks for developing this useful tool. I discovered DMeta while searching for a pre-commit hook that can automatically remove metadata from Excel files before committing them to Git. I couldn't find an existing pre-commit hook for this purpose.

It would be helpful if DMeta had a pre-commit hook. Would you consider implementing this by adding a .pre-commit-hooks.yaml file? Possible options include:

  1. Adding it to the same repository (openscilab/dmeta)
  2. Creating a separate repository (e.g., openscilab/pre-commit-hook-dmeta)
  3. Hosting it in a third-party repository

For pre-commit users, option 1 seems to be the most straightforward choice. However, if you prefer not to add a tool-specific file to the repository or not to maintain a separate tool-specific repository, option 2 or 3 would be reasonable.

See also:

@AHReccese AHReccese self-assigned this Jan 22, 2025
@AHReccese
Copy link
Member

AHReccese commented Jan 22, 2025

@tueda
Thank you for your insightful suggestion! I really appreciate you proposing the idea of adding a pre-commit hook to automatically remove metadata from Excel files before committing them to Git. It's a very useful feature. We're also planning to broaden the supported formats, so it can be extended beyond Microsoft Office files.

I'll certainly take this into consideration for one of the upcoming milestones of DMeta.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants