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

chore: disable registering private repositories #1159

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

rdimitrov
Copy link
Member

@rdimitrov rdimitrov commented Oct 10, 2023

The following PR updates:

  • Disables listing (therefore being able to register) private repositories.
  • Disables processing private repositories when trying to register a repo
  • Grouped a few prints into a single line for repo register

Fixes #1083

PS. I guess we can merge this right before the MVP if we want to keep using private repos for development.

@rdimitrov rdimitrov requested a review from JAORMX October 10, 2023 08:34
@rdimitrov rdimitrov self-assigned this Oct 10, 2023
JAORMX
JAORMX previously approved these changes Oct 10, 2023
cmd/cli/app/repo/repo_register.go Outdated Show resolved Hide resolved
@rdimitrov rdimitrov requested a review from JAORMX October 10, 2023 12:39
Signed-off-by: Radoslav Dimitrov <radoslav@stacklok.com>
@rdimitrov rdimitrov merged commit d86b9f6 into mindersec:main Oct 10, 2023
12 checks passed
@rdimitrov rdimitrov deleted the skip-private branch October 10, 2023 14:05
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

Successfully merging this pull request may close these issues.

Do not register private repos
2 participants