Skip to content
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

Open
dmitryvinn opened this issue Jul 16, 2019 · 4 comments
Open

Links to Private Repos in Public Repos #369

dmitryvinn opened this issue Jul 16, 2019 · 4 comments

Comments

@dmitryvinn
Copy link
Contributor

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:

  1. Do nothing (i.e. leave things as is)
  2. Remove linking to the private repos (i.e. instead of src-d/issues-general, just use src-d/issues-general)
  3. Do not put links to the private repos at all
  4. Open all private repos referenced in public repositories (probably, not an option for most)

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

@dpordomingo
Copy link
Contributor

dpordomingo commented Jul 16, 2019

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.
But yes, we should aware of this limitation in order to provide meaningful info to external contributors or users.

@dmitryvinn
Copy link
Contributor Author

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?

@smola
Copy link
Contributor

smola commented Jul 17, 2019

Some comments about specific cases:

  • src-d/issues-general and src-d/issues-infrastructure are not even used anymore, so we should replace them with current links.
  • for links to backlog, I always prefer to add references to issues from the backlog epics, this creates a backreference in the public issue that is only visible to people with permissions.e
  • other cases in issues are trickier, and just not having any reference at all would be worse for us.

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.

@dmitryvinn
Copy link
Contributor Author

I think it is a great idea - let's append [private link] to the private links!
Unless there are any objections or other ideas, I will be closing the issue.

dmitryvinn added a commit to dmitryvinn/guide that referenced this issue Aug 31, 2019
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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants