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

Already on GitHub? Sign in to your account

Allows only repo owners to change repo head (based on a config variable) #29

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants

fabn commented Aug 24, 2011

Patch attached. I think this could be useful sometime.

sorry but the usage message a few lines above clearly says the full refs/heads/foo is expected and this is intended that way.

Why lose the flexibility for someone to want to set-head to refs/foo/bar -- not all refs need be in refs/heads and refs/tags

fabn commented Aug 25, 2011

I've updated the code in order to allow both partial and full matches for references. Current code check if $2 is a valid full reference. If not it checks for a match in refs/heads and then in refs/tags.

In this way you can use both syntaxes.

What do you think?

Hi,

Thanks...

Actually, all ADCs are supposed to be site-local anyway, which is why they are in contrib. And as for flexibility, the next person may want this at the level of each repo, which cannot be covered by what you have.

I'd rather leave it alone, if you don't mind.

regards

sitaram

@sitaramc sitaramc closed this Nov 16, 2011

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