-
Notifications
You must be signed in to change notification settings - Fork 101
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
Links to Private Repos in Public Repos #369
Comments
More examples of public content linking private content:
I see the problem, from the external visitor, but the pro is that it increases and enhances internal communication. Removing such links will make more difficult to find that info. |
Thank you for adding more examples! Yeah, I agree that it streamlines work for internal contributors; hence, I mentioned option #1 as a valid choice, but wanted to know what others think about it. Let's see if there are any other opinions on this one - if nothing changes, we will just stick to option 1 for now and try to avoid adding more links to the private repos in the future. Does it sound reasonable? |
Some comments about specific cases:
For the guide, maybe we could always highlight with a [private link] suffix any link that leads to private repositories. This way at least it's obvious to the reader that it's an internal resource. |
I think it is a great idea - let's append [private link] to the private links! |
As decided in src-d#369, adding private-link suffix to internal URLs. Additionally, remove a non-existent link to general issues Signed-off-by: Dmitry Vinnik <[email protected]>
Throughout the public repository, I found quite a few links to the private repos which obviously greeted me with a 404 page from Github. Personally, I do not think it is the best experience for those who are outside of the organization.
I was thinking about potential options:
Personally, option 2 sounds fairly decent, but if we consider that this guide is primarily intended for internal use (i.e. those with access to all private repos), then option 1 is good enough.
Note: Example of public repos with private links: https://github.com/src-d/guide/blob/master/engineering/teams/methodology.md
The text was updated successfully, but these errors were encountered: