Skip to content

user invitations: exclude 'user public repos', forks, archived repos from collab discovery

Warren Gifford requested to merge sg/better-collab into main

Created by: slimsag

collaborator discovery prior to this commit came from the user's repos which was including:

  • 'user public repos' which apparently "are not repos owned by this user, just ones they are interested in."
  • forks / archived repos, obviously these are bad sources to find collaborators from.

Should help improve the results that e.g. Tomas got here: https://sourcegraph.slack.com/archives/C01LZKLRF0C/p1647115816296369

Signed-off-by: Stephen Gutekanst [email protected]

Test plan

Manually tested, we have unit tests covering much of the logic in this endpoint but not mocking out the repository listing behavior. May be worth investing in later if this feature remains worthwhile / after the experiment is over.

Merge request reports

Loading