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

Document decision to forgo repo creation #459

Merged
merged 2 commits into from
Jul 12, 2021
Merged

Conversation

palemtnrider
Copy link
Contributor

What changed?
Added a new ADR documenting the decisions to forgo repo creation

Why?
We have encountered many environments that prohibit repo creation in their git organization

How did you test it?
N/A

Release notes
N/A

Documentation Changes
N/A

Copy link
Contributor

@jrryjcksn jrryjcksn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM


## Context

In the initial pre-alpha version of weave-gitops and WKP, we created git repositories for the end-user. However, we encounter more instances where customers don't permit developers, application or platform SREs to create repositories in the customer's organization. While this is a convenient feature, we've decided the effort and support required are better focused elsewhere in the application.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: I don't think the second sentence scans very well. Maybe: "However, we have encountered more (or many)" or just "However, we have encountered instances..."

@palemtnrider palemtnrider merged commit 8815bcc into main Jul 12, 2021
@palemtnrider palemtnrider deleted the adr-repo-handling branch July 12, 2021 21:49
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.

None yet

2 participants