-
Notifications
You must be signed in to change notification settings - Fork 61.1k
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
Pre-receive hooks/$GITHUB_VIA #1082
Comments
Thanks for opening this issue. A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines. |
Sorry, not sure how to respond to that! Could you let me know what you need more clearly please? Edit: The weird image-only comment got deleted. Leaving this here so that context doesn't get destroyed. |
The Thank so much for letting us know about this @blastedt! I'll have some folks internally take a look at this and see what updates we might be able to make 💛 In the meantime, I'll add a |
Thanks Janice, I appreciate it. Tim |
This comment has been minimized.
This comment has been minimized.
Hey Matt, thanks for closing this out! The docs look super helpful now. Tim |
👋🏻 Thanks for creating this issue, @blastedt 🙏🏻 And thanks for your patience while I did some research and prioritized the changes. Let us know if you see any further opportunities to improve this or other documentation 🚀 |
What article on docs.github.com is affected?
https://github.com/github/docs/blob/main/content/admin/policies/creating-a-pre-receive-hook-script.md
What part(s) of the article would you like to see updated?
The environment variable $GITHUB_VIA has a list of possible values.
slumlord
,stafftools
, etc are unclear)blob#save
which isn't listed in the table at all.git push
in their local terminal) will not have$GITHUB_VIA
set, and to list what other situations this variable will not be set.Additional information
I would make this change myself but I honestly am having trouble determining the intended behavior! A docs update would be a great help to me.
The text was updated successfully, but these errors were encountered: