-
Notifications
You must be signed in to change notification settings - Fork 54
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
📖 Content organization #1324
📖 Content organization #1324
Conversation
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com>
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
f2e586d
to
8cbd83b
Compare
575d580
to
a552d87
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1324 +/- ##
=======================================
Coverage 76.17% 76.17%
=======================================
Files 41 41
Lines 2409 2409
=======================================
Hits 1835 1835
Misses 401 401
Partials 173 173
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
4bbe7df
to
392d279
Compare
392d279
to
f8879bb
Compare
c15c145
to
eccbe4b
Compare
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.
Only nits. I have some concerns about the evident need to hide ToC (since I don't know why), but it doesn't rise to the level of challenging the move.
Separately, we need some love on the contents. In general I've tried to flag comments about existing content in a clear way so we can keep this PR focused on the portal organization / format instead. I've identified one existing editorial issue I think we can easily solve though, and a potential netlify problem when previewing bulleted lists that I ran into with semver template docs in the past.
eccbe4b
to
45ee3e2
Compare
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com>
45ee3e2
to
0ffea04
Compare
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
Description
Closes #1305
This PR takes a first stab at organizing the content in docs/drafts into some kind of coherent structure.
There's still much to do in terms of making the documents more uniform and prettying them up.
Reviewer Checklist