-
Notifications
You must be signed in to change notification settings - Fork 71
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
Refactor docs #296
base: main
Are you sure you want to change the base?
Refactor docs #296
Conversation
Signed-off-by: Natalie Arellano <narellano@vmware.com>
Signed-off-by: Natalie Arellano <narellano@vmware.com>
Maintainers, As you review this RFC please queue up issues to be created using the following commands:
Issues(none) |
Signed-off-by: Natalie Arellano <narellano@vmware.com>
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.
I like the general thrust of this proposal.
Signed-off-by: Natalie Arellano <narellano@vmware.com>
Signed-off-by: Natalie Arellano <narellano@vmware.com>
Signed-off-by: Natalie Arellano <narellano@vmware.com>
Signed-off-by: Natalie Arellano <narellano@vmware.com>
@natalieparellano Do you want to shepherd this, or would you prefer if I did it? |
@AidanDelaney if you could that'd be great :) |
@natalieparellano will update this RFC with some examples of what the sidebar could look like |
text/0000-re-org-docs.md
Outdated
|
||
- Define the target persona: all personas, including project contributors, can benefit from this. | ||
- Explaining the feature largely in terms of examples. | ||
- See the example [branch](https://github.com/buildpacks/docs/tree/refactor) of github.com/buildpacks/docs and abbreviated tree with notes below. |
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.
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.
WDYT @AidanDelaney?
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.
I like this. It's much more straightforward than what I was thinking and thus more achievable.
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 :)
Moved to voting, closing Thurs Feb 8 :) |
text/0000-re-org-docs.md
Outdated
|
||
Why should we *not* do this? | ||
- It's a lot of work | ||
- Changing the structure of the docs could be surprising for users who are already familiar with its content |
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.
It also potentially breaks SEO and links to the documentation outside of us. Will we redirect to the homepage or other pages?
Land the RFC describing how to refactor our documentation
I'm not sure why this hasn't been merged (as it's already been implemented 🙃) - will take a look. |
Readable