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

Revised TechDoc Analysis Tools #229

Merged
merged 10 commits into from
May 31, 2024
Merged

Conversation

dwelsch-esi
Copy link
Collaborator

Updated READMEs, how-to, and templates for CNCF tech docs analyses. Changes reflect the changes and additions to the tech docs analysis project since November 2023. Reorganized the tech docs directories.

@dwelsch-esi
Copy link
Collaborator Author

@chalin @thisisobate @krook @nate-double-u I've updated the tech-docs analysis process documentation, including the howto, templates, and READMEs. I've also re-arranged the directory structure to reflect naming changes we agreed on (for example "analysis" instead of "assessment", and to better contain the analysis materials. Please review. This is a first draft and I'm sure there will be changes -- I need your feedback!

## Office hours

The CNCF tech docs team generally holds office hours on the [fourth Wednesday of every month at 8am Pacific time](https://tockify.com/cncf.public.events/monthly?search=CNCF%20Tech%20Writers%20Office%20Hours).
The CNCF tech docs team holds office hours on the [fourth Wednesday of every month at 8am Pacific time](https://tockify.com/cncf.public.events/monthly?search=CNCF%20Tech%20Writers%20Office%20Hours).
Copy link
Collaborator

Choose a reason for hiding this comment

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

We had agreed to call these meetings something other than Office Hours. Just a note in passing. We can update in a followup PR.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@chalin chalin May 14, 2024

Choose a reason for hiding this comment

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

Rename to techdocs-assistance.md, or maybe just assistance.md given that this entire repo is about techdocs :)

Copy link
Member

Choose a reason for hiding this comment

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

I agree, and think I prefer just assistance.md here.

TechDoc-Assistance-Program.md Outdated Show resolved Hide resolved
analysis/0001-contour.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Thanks @dwelsch-esi. See inline comments.

Also, I suggest:

  1. Reverting the rename of the docs folder to resources
  2. Renamed the analysis folder to analyses
  3. Move & rename /analysis/analysis-tools to /resources so that analyses contains only analyses.
  4. Atm, there should be only templates in the (new) /resources: move the other pages to be under docs:
    image

TechDoc-Assistance-Program.md Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

Let's add a README.md file to the analysis-tools directory to build an index in.

@chalin
Copy link
Collaborator

chalin commented May 14, 2024

@dwelsch-esi - after addressing the suggested changes, pls rebase, resolve conflicts, and push back the PR (otherwise rebasing might cause the suggestions to be lost).

Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

More copyedits

TechDoc-Assistance-Program.md Outdated Show resolved Hide resolved
TechDoc-Assistance-Program.md Outdated Show resolved Hide resolved
TechDoc-Assistance-Program.md Outdated Show resolved Hide resolved

A technical writer (on CNCF staff or on contract) analyzes the documentation. Based on the standards developed by the CNCF Tech Docs program, the writer:

1. Estimates the maturity level of the documentation compared to the current or desired maturity level of the software project, based on a number of criteria in three categories, using a rubric developed by CNCF. The categories are:
Copy link
Collaborator

Choose a reason for hiding this comment

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

A bit of a run-on sentence, consider revising.

TechDoc-Assistance-Program.md Outdated Show resolved Hide resolved
TechDoc-Assistance-Program.md Outdated Show resolved Hide resolved

Community members work on the issues created in the previous phase. Ideally, tech writers are available to advise contributors and edit work, especially if the contributing community members are not trained technical writers. Remember that the training courses in Phase 0 are available to prepare contributors with general knowledge of the process.

We know that recruiting contributors to write documentation can be difficult. While this is largely the responsibility of the project leadership, CNCF is actively working on ways to encourage doc contributions. For example, creating a backlog of time-bounded issues is an attempt to lower barriers, both psychological and logistical, to documentation creation and maintenance.
Copy link
Collaborator

Choose a reason for hiding this comment

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

@nate-double-u - Do we want to mention mentorship opportunities twice(?) a year?

TechDoc-Assistance-Program.md Outdated Show resolved Hide resolved
TechDoc-Assistance-Program.md Outdated Show resolved Hide resolved

Measurement is vital to any process improvement effort, and this is currently the weakest point of the TechDoc Assistance Program. We do no offer any tools to measure the impact of doc improvements, although we can assure you that they are profound. Projects are encouraged to collect metrics on documentation effectiveness.

As a basic gauge the effectiveness of the documentation effort, projects can track the issue backlog. This can provide a guide to progress on leveling the documentation maturity to that of the project.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hmm, I'm not sure about this paragraph. How about (or just drop the paragraph IMHO):

Suggested change
As a basic gauge the effectiveness of the documentation effort, projects can track the issue backlog. This can provide a guide to progress on leveling the documentation maturity to that of the project.
Projects can also track the issue backlog as a means of measuring the increased documentation maturity.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Agreed.

Copy link
Collaborator Author

@dwelsch-esi dwelsch-esi left a comment

Choose a reason for hiding this comment

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

Reconciling last few commits plus in-toto assessment commit.

docs/criteria.md Outdated
Copy link
Member

Choose a reason for hiding this comment

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

I think that the docs/criteria.md and docs/howto.md files should be housed in the new resources folder.

Copy link
Member

@nate-double-u nate-double-u May 16, 2024

Choose a reason for hiding this comment

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

I take it back. I think we should create an analysis folder in docs, which would contain the howto and criteria files, and the resources folder, to parallel how we've organized the localization folder.

Copy link
Member

Choose a reason for hiding this comment

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

this leaves the top level analysis folder as a listing of completed assessments.

@nate-double-u
Copy link
Member

I'll fix the DCO issue on squash.

@nate-double-u
Copy link
Member

nate-double-u commented May 16, 2024

Let's review and agree on this folder stucture, and if we agree merge, then in a follow up PR do the assessment to analysis folder rename.

@dwelsch-esi
Copy link
Collaborator Author

The howto.md file is now in docs/analysis, so you can't automatically view the diff in the PR files view. I've pasted them here so you can just chuck them into your favorite diff tool if you like, though they're different enough that I suspect you'll just want to review the new file standalone. analysis-howto.md is the new file.

analysis-howto.md
main-howto.md

README.md Outdated

The TechDocs team can assist CNCF projects analyze and improve their documentation. For details, see the TechDocs [assistance-program][].

[assistance-program]: ./techdoc-assistance.md
Copy link
Member

Choose a reason for hiding this comment

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

Where should this link now?

Copy link
Member

Choose a reason for hiding this comment

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

nvm, found it :)

@nate-double-u
Copy link
Member

I've gone through and corrected most links (except those in the templates section, @chalin, we may need to add an exclusion for the check), and have reformatted some tables and other things.

Copy link
Member

@nate-double-u nate-double-u left a comment

Choose a reason for hiding this comment

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

/lgtm

but I'll let @chalin do a last once over before merging.

@chalin
Copy link
Collaborator

chalin commented May 28, 2024

Thanks all! Was OOO last week, and was catching up until now. I'll review this tomorrow morning as soon as I can.

@nate-double-u - who needs to fix the DCO-check failure?

@nate-double-u
Copy link
Member

I'll fix that on squash/merge.

@chalin
Copy link
Collaborator

chalin commented May 31, 2024

Rebased from main.

@nate-double-u
Copy link
Member

thansk @chalin, i was just looking to do that (GitHub UI was saying out of date, start over, and it was confusing 😅)

assistance.md Outdated Show resolved Hide resolved
@chalin
Copy link
Collaborator

chalin commented May 31, 2024

@nate-double-u - with all GH actions passing, overall I agree that this looks good. I'd suggest we merge, and then I can make a pass or two to move things around etc. WDYT? As mentioned before the DCO check is failing, but you said that you'd fix it when merging 👍.

Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Great improvement!
Next time let's do this via a series of smaller PRs.
LGTM. I'll make a pass or two after.

Omphaloskepsis and others added 10 commits May 31, 2024 12:13
Added new analysis templates.
Renamed directories to better reflect their contents.
Rewrote README files.

Signed-off-by: David Welsch <dwelsch@expertsupport.com>
Signed-off-by: Patrice Chalin <pchalin@gmail.com>
Signed-off-by: David Welsch <dwelsch@expertsupport.com>
Signed-off-by: Patrice Chalin <pchalin@gmail.com>
… templates. Reorganized tech-docs directory structure.

Signed-off-by: David Welsch <dwelsch@expertsupport.com>
Signed-off-by: Patrice Chalin <pchalin@gmail.com>
Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>
Signed-off-by: Dave Welsch <116022979+dwelsch-esi@users.noreply.github.com>
Signed-off-by: Patrice Chalin <pchalin@gmail.com>
Signed-off-by: David Welsch <dwelsch@expertsupport.com>
Signed-off-by: Patrice Chalin <pchalin@gmail.com>
Signed-off-by: David Welsch <dwelsch@expertsupport.com>
Signed-off-by: Patrice Chalin <pchalin@gmail.com>
…y structure chnages included.

Signed-off-by: David Welsch <dwelsch@expertsupport.com>
Signed-off-by: Patrice Chalin <pchalin@gmail.com>
Signed-off-by: David Welsch <dwelsch@expertsupport.com>
Signed-off-by: Patrice Chalin <pchalin@gmail.com>
Signed-off-by: Nate W <natew@cncf.io>
Signed-off-by: Patrice Chalin <pchalin@gmail.com>
Signed-off-by: Patrice Chalin <pchalin@gmail.com>
@nate-double-u nate-double-u merged commit 81199a2 into cncf:main May 31, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants