-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
Migrate documentation: Remove docs from etcd-io/etcd #12660
Conversation
I've tried a first pass at removing the documentation as per issue etcd-io/website#97. Could you look at this PR to ensure that I've not broken any of the documentation generation tools, and that the release scripts still work as intended? |
I think that you can move. If we discover broken scripts, we will fix them when we will attempt the release. |
Thanks @ptabor, in that case, I'll mark this PR "ready for review" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Let's keep it up for review for couple of more days before we merge. Thanks!
I'm for merging. |
@nate-double-u please resolve conflicts when you get a chance :) |
@nate-double-u the commit 2 doesn't sound right that I think you have created to fix the merge conflict. You just need to rebase not overwrite. Thanks! |
b73441d
to
2511790
Compare
Leaving docs, files, etc that are required for generated documentation fixes etcd-io/website#97 Signed-off-by: Nate W <4453979+nate-double-u@users.noreply.github.com>
2511790
to
2afa5ff
Compare
@spzala it's rebased now, thx :) |
@nate-double-u cool,thanks :) |
Thanks everyone! |
Removing Documentation from etcd-io/etcd as a part of migrating it to the website.
fixes etcd-io/website#97
Do not merge before related PR: etcd-io/website#99
Affected PRs (these may have to be redirected towards etcd-io/website):