Skip to content
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

All-in-one doc template idea #16

Merged
merged 3 commits into from
Aug 6, 2021
Merged

All-in-one doc template idea #16

merged 3 commits into from
Aug 6, 2021

Conversation

quaid
Copy link
Member

@quaid quaid commented Jul 17, 2021

  • This includes several sections usable for one or more document types: how-to, article, FAQ, and so forth.
  • A writer picks one of the templates and discards the rest of the markup.
  • The templates are displayed as code blocks by indenting 4 characters; this makes the markup visible on the rendered page. To use the template, the 4 characters must be removed.
  • All of this are just some ideas of how to create and maintain a single template.

Signed-off by: Karsten Wade kwade@redhat.com quaid@iquaid.org

- This includes several sections usable for one or more document types: how-to, article, FAQ, and so forth.
- A writer picks one of the templates and discards the rest of the markup.
- The templates are displayed as code blocks by indenting 4 characters; this makes the markup visible on the rendered page. To use the template, the 4 characters must be removed.
- All of this are just some ideas of how to create and maintain a single template.

Signed-off by: Karsten Wade <kwade@redhat.com> <quaid@iquaid.org>
@quaid quaid linked an issue Jul 17, 2021 that may be closed by this pull request
@sesheta sesheta added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jul 17, 2021
@quaid quaid added this to Proposed content in Community Handbook Jul 17, 2021
@quaid quaid moved this from Proposed content to Content in progress in Community Handbook Jul 17, 2021
@@ -0,0 +1,147 @@
<!-- Title, author, date block standard in all files -->
# Title (ex: How to Use `o1-clone` tool)
Copy link
Contributor

@margarethaley margarethaley Jul 19, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would just suggest making "Use" lowercase instead, since we want the titles to be in sentence case. Other than that, I think it looks good!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, maybe this document should go in the contributing folder?

Community Handbook automation moved this from Content in progress to Review in progress Jul 19, 2021
Signed-off by: Karsten Wade <kwade@redhat.com> <quaid@iquaid.org>
- This makes sense to have an easy directory with the files for contributing.
- We'll have to link some/all of it into the Community Handbook, as well.

Signed-off by: Karsten Wade <kwade@redhat.com> <quaid@iquaid.org>
@quaid quaid requested a review from margarethaley July 24, 2021 01:34
Copy link
Contributor

@margarethaley margarethaley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! So sorry for the delay, I didn't see the updates until just now!
/lgtm

Community Handbook automation moved this from Review in progress to Approved, needs to be added to Table of Contents Aug 6, 2021
@sesheta sesheta added the lgtm Indicates that a PR is ready to be merged. label Aug 6, 2021
@sesheta
Copy link
Member

sesheta commented Aug 6, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: margarethaley

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sesheta sesheta added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 6, 2021
@margarethaley margarethaley merged commit 729825b into main Aug 6, 2021
Community Handbook automation moved this from Approved, needs to be added to Table of Contents to Done Aug 6, 2021
@quaid quaid deleted the quaid-template-idea branch August 6, 2021 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
Development

Successfully merging this pull request may close these issues.

Article template
3 participants