Re: Git not commiting anything if file created and "git add"ed in pre-commit hook is the only file in the staging area
From: Peter Hunkeler <hidden>
Date: 2021-10-30 13:21:33
Pardon my ignorance, but I'm unlear as to how to proceed further with my issue. What is the proper process to report a bug, and get a consense whether it is accepted or rejected? Regards Peter Am 28.10.2021 um 14:08 schrieb Peter Hunkeler:
Am 28.10.2021 um 00:07 schrieb brian m. carlson:quoted
I should point out here that it isn't intended for pre-commit hooks to be used this way; they're intended to verify that the commit meets some standards, not to modify it, although it is of course possible to do.I can accept that comment. However: - wouldn't you agree that git should work consistently? It does not in this case. If there is anything to be commited in the index, then the "git add" from within the pre-commit hook *is* respected in this commit. If there is *nothing* to be commited, except from what was added by the pre-commit exit, then it is ignored *for this commit*, but it is added and will be commited next time. This is inconsistent behaviour. - if the decision will be *not* to allow adding from within a pre-commit hook, then the "git add" should be rejected. And the documentation should say so. I'll have to understand what all the comments from you and others mean (still a git newbee).quoted
In general, you want to avoid adding automatically generated files to your repository. That tends to bloat the repository needlessly and is a great way to lead to frequent, hard-to-resolve merge conflicts.I'm trying to use git to keep track of changes to my website, which uses some framework. The problem is: Some changes modify filesystem entities, only, while other changes modify the content of some mysql tables. So I need to pack an unload of those tables into the commit. I can do this manually (and forget every now and then :-), or I thought the pre-commit hook would be a good place to automate this. And, yes, it may well crete merge conflicts. I understand the pre-commit hook is a local thing, and I have to make sure the same is active in all repositories. Definitely not something to use in a widely shared project. Thanks a lot Peter