-
Notifications
You must be signed in to change notification settings - Fork 508
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
removing registry page #802
Conversation
Codecov Report
@@ Coverage Diff @@
## master #802 +/- ##
=======================================
Coverage 57.44% 57.44%
=======================================
Files 76 76
Lines 2437 2437
=======================================
Hits 1400 1400
Misses 913 913
Partials 124 124 Continue to review full report at Codecov.
|
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.
Approved but please check:
Development.md
- links registryReadme.md
- links registry- also
/design/proxy.md
links this file on multiple places
I think we should at least remove the links to registry.md from other files. |
@marpio @michalpristas thanks for catching all the registry references that I missed 😄 I made a lot of changes that I think removes everything related to the registry. I think things got clearer too Can you take another look when you have time? |
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.
Perfect!
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.
Awesome!
Thanks @marpio and @michalpristas for helping me so much here! |
What is the problem I am trying to address?
The docs still had a page referring to the registry. We've decided that we're not trying to build a registry right now (see #772)
How is the fix applied?
I just removed the
docs/content/design/registry.md
pageMention the issue number it fixes or add the details of the changes if it doesn't has a specific issue.
Fixes #732