-
Notifications
You must be signed in to change notification settings - Fork 419
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
Move git-sync to a new repo of its very own #1
Conversation
There is a process for new repos now https://docs.google.com/document/d/1ugAd9Zj-jW3YHdrNVdktmvDMEWtChPqyGHfkwWdQ3zo/edit#heading=h.bv4kogapeb2x |
The process doesn't apply well to code that is moving from and existing On Aug 20, 2016 9:44 PM, "Mike Danese" notifications@github.com wrote:
|
Yeah, this falls under the "existing" section:
That said, I have some significant followups to this, so a rubber stamp is all that is required, really. |
Conveniently you left out the (except contrib/)
I'm not sure how this is an improvement over what we currently have but I'm not familiar enough with the new process (and I don't even know if it's been enacted yet (seems like it hasn't since https://github.com/kubernetes-incubator is empty)). I give this the rubber stamp. |
the "except contrib/" is about the existing On Mon, Aug 22, 2016 at 9:27 AM, Mike Danese notifications@github.com
|
The quote also says main repo, which this code was not in. When it was in the main repo, it was in contrib/, |
OK, then I have no idea what the "except contrib" means in this context. I On Mon, Aug 22, 2016 at 3:42 PM, Mike Danese notifications@github.com
|
SGTM |
This just imports the existing state of contrib/git-sync into this new repo and scrubs paths for "contrib". I have some further work I want to do on this sidecar, so I thought it was time to break it out.