add hook to reject certain commits from being pushed to the repository

[larsxschneider]

This can be a useful pre-receive hook if you rewrote the history of a repository and you want to ensure nobody pushes the old commits again.

[IAmHughes]

This looks great @larsxschneider. I had a suggestion, but I'm okay with leaving things as they are too so I'm approving.

Changes requested were resolved

[mzzmjd]

Hi @larsxschneider,
This is a good idea, I am just thinking aloud about the administrator actions required ..... 🤔

Do we store this pre-receive hook script in the repository that is protected?
That way the owner of the repository can add / remove the required commits But as an admin I have to set up the pre-receive hook for them.

Was just wondering if the repo could have a list of the commits in a text file (using a standard naming convention).
There could then be a single pre-receive hook, across the whole instance, that could then use the api to get the contents of the file. Providing that they give a designated whole instance api user access to the repo. Or is that making it a bit more complicated and open to failure?

Pete (from Sheffield)

[larsxschneider]

@mzzmjd Thanks for you feedback Pete! I get your point and that makes total sense. I'll add that option 👍

[Aa88s]

Yes pull requests looks much better