Skip to content

OSDOCS-18373 HE-3 CQA B2: CQA editorial improvements for NFD - #117872

Open
cbippley wants to merge 1 commit into
openshift:mainfrom
cbippley:OSDOCS-18373-b2-split2
Open

OSDOCS-18373 HE-3 CQA B2: CQA editorial improvements for NFD#117872
cbippley wants to merge 1 commit into
openshift:mainfrom
cbippley:OSDOCS-18373-b2-split2

Conversation

@cbippley

@cbippley cbippley commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Version(s): 4.20

Issue: https://redhat.atlassian.net/browse/OSDOCS-18373

Link to docs preview: https://117872--ocpdocs-pr.netlify.app/openshift-enterprise/latest/hardware_enablement/psap-node-feature-discovery-operator.html or review guide below.

QE review: NA, CQA

  • QE has approved this change.

Additional information:

This is part 2 of 2, split from #117336 for easier review. No merge conflicts with part 1 (#117871) — can be merged in either order.

Summary

  • Rewrites abstracts in nfd-rules-about.adoc and nfd-rules-using.adoc
  • Fixes wording (labelling → labeling, tightens sentences)
  • Replaces psap-using-node-feature-discovery-operator.adoc include with new nfd-using-operator-overview.adoc
  • Rewrites the assembly intro text

Test plan

  • Verify the assembly renders correctly with the updated modules
  • Verify abstract text appears properly in each module

🤖 Generated with Claude Code

Rewrite abstracts, fix wording in rules modules, and replace the
using-operator module with a clearer NFD Operator overview.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-ci openshift-ci Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Aug 12, 2026
@ocpdocs-previewbot

Copy link
Copy Markdown

@openshift-ci

openshift-ci Bot commented Aug 12, 2026

Copy link
Copy Markdown

@cbippley: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@cbippley

Copy link
Copy Markdown
Contributor Author

Review Guide

4 files changed | +26 lines | -8 lines

Changes

# Section What changed
1 Assembly intro Rewrote abstract from passive ("Learn about…") to active ("You can use…")
2 NFD Operator overview New module replacing psap-using-node-feature-discovery-operator include
3 About NodeFeatureRule Consolidated 2 paragraphs into 1 abstract sentence
4 Using NodeFeatureRule Added abstract, fixed "labelling" → "labeling", tightened wording

Preview Links

# Section Link
1 Assembly intro View
2 NFD Operator overview View
3 About NodeFeatureRule View
4 Using NodeFeatureRule View

@cbippley

Copy link
Copy Markdown
Contributor Author

/label merge-review-needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.20 branch/enterprise-4.21 branch/enterprise-4.22 branch/enterprise-5.0 merge-review-needed Signifies that the merge review team needs to review this PR size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants