-
Notifications
You must be signed in to change notification settings - Fork 60.6k
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
Statement about visibility of custom properties is misleading or incorrect #35819
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. |
@svrnm Thank you for raising this issue! I'll get this triaged for review ✨ Our team will provide feedback regarding the best next steps for this issue - thanks for your patience! 💛 |
Thanks for opening an issue! We've triaged this issue for technical review by a subject matter expert 👀 |
@svrnm Thank you so much for your patience while our SME team reviewed! ✨ They are aligned with you, and wanted to relay the following:
Would you be willing to submit a PR to update that statement to align with our SME's suggestion? 💛 |
Thanks @nguyenalex836. I don't see how this sentence is different to the existing one? It still states can be misread that explicitly set reading permissions are required to view them, however for a public repository it means EVERYONE can view them. So I would expect a wording like
To add this for context: I was considering to add information as a custom property to a repository that is for internal purpose only, and in some cases might even be considered sensitive, so as it stands today custom properties should not be used for that, and the documentation should make this entirely clear. |
Code of Conduct
What article on docs.github.com is affected?
https://docs.github.com/en/organizations/managing-organization-settings/managing-custom-properties-for-repositories-in-your-organization
What part(s) of the article would you like to see updated?
By simply reading this statement I would assume that if I set a custom property on a repository it is only visible to people with assigned permissions, but in the case of a public repository everyone can see the custom property. I have verified it with the following steps:
foo
)foo
with valuebar
foo
is visible although I am not logged inEither this is misleading, since everyone has read permissions on a public repository in some way, or this is incorrect, and the statement needs to be updated, or this is unintended and needs to be fixed.
Thanks
Additional information
The text was updated successfully, but these errors were encountered: