Skip to content

Added guidelines about where links are allowed - #596

Merged
IngridT1 merged 5 commits into
redhat-documentation:mainfrom
lburcher7:where_links_are_allowed
Sep 16, 2026
Merged

IngridT1 merged 5 commits into
redhat-documentation:mainfrom
lburcher7:where_links_are_allowed

Conversation

@lburcher7

Copy link
Copy Markdown
Contributor

Added guidance about links not being allowed in short descriptions and concept modules. Two files were updated.

Issue: #594

@coderabbitai

coderabbitai Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 70c34e53-4d12-47b0-8dbd-3a3dfa21c756

📥 Commits

Reviewing files that changed from the base of the PR and between 4171f2b and 257451b.

📒 Files selected for processing (1)
  • supplementary_style_guide/style_guidelines/links.adoc

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting


📝 Summary

Summary by CodeRabbit

  • Documentation
    • Added guidance on where inline links may be used in concept, procedure, reference, and short-description topics.
    • Clarified that inline links are not compatible with short descriptions, alongside bulleted lists and multiple paragraphs.

Walkthrough

The style guide now defines where inline links are allowed. It also identifies inline links as incompatible with short descriptions.

Changes

Inline link guidance

Layer / File(s) Summary
Define inline-link rules
supplementary_style_guide/style_guidelines/links.adoc, supplementary_style_guide/style_guidelines/structure.adoc
The links guide defines inline-link rules for concept, procedure, and reference topics. It directs essential or supplemental links to “Related Links” when applicable. The structure guide prohibits inline links in short descriptions.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Suggested reviewers: ingridt1, neal-timpe, sheriff-rh

Merge Risk: ⚪ Minimal · up to 4171f

The updated guidance consistently restricts inline links, with no substantiated issue blocking merge.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the added guidance about where links are allowed.
Description check ✅ Passed The description explains the change, identifies the two affected topics, and includes the related issue link. The optional Additional information section is not included, but the required information …
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

Comment thread supplementary_style_guide/style_guidelines/links.adoc Outdated
Comment thread supplementary_style_guide/style_guidelines/links.adoc Outdated
Comment thread supplementary_style_guide/style_guidelines/links.adoc Outdated
Comment thread supplementary_style_guide/style_guidelines/links.adoc Outdated

@IngridT1 IngridT1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM with the small changes.

@sheriff-rh sheriff-rh left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Changes LGTM, thank you!

@IngridT1

Copy link
Copy Markdown
Collaborator

Neal Timpe and Andy Taylor approved the PR.

@IngridT1 IngridT1 closed this Sep 15, 2026
@IngridT1 IngridT1 reopened this Sep 16, 2026
@IngridT1
IngridT1 merged commit 484b46d into redhat-documentation:main Sep 16, 2026
3 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