Skip to content

[RHIDP#12789]: Fix CQA PR comment rendering issues#1952

Merged
themr0c merged 1 commit into
redhat-developer:mainfrom
themr0c:RHIDP-12789-fix-cqa-pr-comment-rendering
Mar 20, 2026
Merged

[RHIDP#12789]: Fix CQA PR comment rendering issues#1952
themr0c merged 1 commit into
redhat-developer:mainfrom
themr0c:RHIDP-12789-fix-cqa-pr-comment-rendering

Conversation

@themr0c

@themr0c themr0c commented Mar 20, 2026

Copy link
Copy Markdown
Member

IMPORTANT: Do Not Merge - To be merged by Docs Team Only

Version(s): N/A (tooling only)
Issue: https://redhat.atlassian.net/browse/RHIDP-12789
Preview: N/A

Summary

Fixes two rendering issues in the CQA GitHub Actions PR comment:

  • Code block wrapping: The cqa.sh output contained triple-backtick fenced code blocks for the "To auto-fix" and "To fix" instructions, causing GitHub to render the entire checklist as a plain code block instead of formatted markdown with checkboxes. Replaced with inline code.
  • Auto-linked issue numbers: CQA #N notation (e.g., CQA #0, CQA #3) caused GitHub to auto-link #N to unrelated issues/PRs. Replaced with CQA-N across all 43 affected files (scripts, Claude skills, docs).

- Replace triple-backtick fenced code blocks with inline code in cqa.sh
  to prevent GitHub from rendering the entire checklist as a code block
- Replace CQA #N notation with CQA-N throughout all scripts, Claude
  skill files, and documentation to prevent GitHub auto-linking #N to
  unrelated issues/PRs

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rhdh-bot

rhdh-bot commented Mar 20, 2026

Copy link
Copy Markdown
Collaborator

@sonarqubecloud

Copy link
Copy Markdown

@themr0c themr0c merged commit e701fc8 into redhat-developer:main Mar 20, 2026
12 checks passed
@rhdh-bot

Copy link
Copy Markdown
Collaborator

Content Quality Assessment Results

CQA Summary Checklist

  • CQA #0: Find Orphaned Modules and Images
  • CQA #0: Verify Directory Structure (_ naming)
  • CQA RHIDP-208 Add missing images #3: Verify Content Type Metadata
  • CQA RHIDP-208-1 Add plugins support matrix #13: Verify Content Matches Declared Type
  • CQA RHIDP-449-1 Support for dark theme in RHDH #10: Verify Titles Are Brief, Complete, and Descriptive
  • CQA Rhidp 449 Customize theme mode #8: Verify Short Description Content Quality
  • CQA #9: Verify Short Description Format
  • CQA RHIDP-449-2 Update attributes file symbolic link #11: Verify Procedure Prerequisites
  • CQA RHIDP-208 Update attributes file #2: Verify Assembly Structure
  • CQA RHIDP-208 Title structure tidy-up #5: Verify Required Modular Elements
    • [MANUAL] modules/shared/proc-mirror-plugins-from-a-catalog-index-to-a-partially-disconnected-environment.adoc: Non-standard block title: .Example
    • [MANUAL] modules/shared/proc-mirror-plugins-from-a-catalog-index-to-a-fully-disconnected-environment.adoc: Non-standard block title: .Example
    • [MANUAL] modules/shared/proc-mirror-plugins-from-a-catalog-index-to-a-fully-disconnected-environment.adoc: Non-standard block title: .Example
    • [MANUAL] modules/shared/proc-mirror-specific-plugins-by-direct-reference.adoc: Non-standard block title: .Example
    • [MANUAL] modules/shared/proc-mirror-plugins-from-a-file.adoc: Non-standard block title: .Example
    • [MANUAL] modules/shared/proc-combine-many-plugin-sources.adoc: Non-standard block title: .Example
    • [-> CQA RHIDP-208 Update attributes file #2 MANUAL] assemblies/extend_installing-and-viewing-plugins-in-rhdh/assembly-install-plugins-from-oci-registries-by-using-custom-certificates.adoc: Assembly contains block titles (only .Additional resources allowed)
    • [MANUAL] modules/shared/proc-export-plugins-in-rhdh.adoc: Non-standard block title: .Example dynamic plugin export with shared and embedded packages
    • [MANUAL] modules/shared/proc-export-plugins-in-rhdh.adoc: Non-standard block title: .Example command to log the default configuration
    • [MANUAL] modules/shared/proc-export-plugins-in-rhdh.adoc: Non-standard block title: .Default scalprum configuration
    • [MANUAL] modules/shared/proc-export-plugins-in-rhdh.adoc: Non-standard block title: .Example scalprum customization
    • [MANUAL] modules/shared/proc-export-plugins-in-rhdh.adoc: Non-standard block title: .Example static and dynamic plugin export
    • [MANUAL] modules/shared/proc-create-a-tgz-file-with-dynamic-packages.adoc: Non-standard block title: .Example commands to build and deploy an HTTP server in {ocp-short}
    • [MANUAL] modules/shared/proc-load-a-plugin-packaged-as-a-javascript-package.adoc: Non-standard block title: .Example secret configuration
    • [MANUAL] modules/shared/proc-example-of-installing-a-custom-plugin-in-rhdh.adoc: Line 16: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-example-of-installing-a-custom-plugin-in-rhdh.adoc: Non-standard block title: .Container images published to Quay.io
    • [MANUAL] modules/shared/proc-display-the-front-end-plugin.adoc: Line 61: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-view-available-plugins.adoc: Line 14: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-search-for-plugins-by-name.adoc: Line 12: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-manage-plugins-by-using-extensions.adoc: Line 16: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-configure-rbac-to-manage-extensions.adoc: Line 23: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-configure-rhdh-local-to-install-plugins-by-using-extensions.adoc: Has 2 H1 titles (should be exactly 1)
    • [MANUAL] modules/shared/proc-install-plugins-by-using-extensions.adoc: Line 18: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-enable-and-disable-plugins-by-using-extensions.adoc: Line 18: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/ref-customizing-entity-page.adoc: Line 11: Non-standard block title: .Input parameters
    • [MANUAL] modules/shared/ref-customizing-and-extending-entity-tabs.adoc: Line 43: Non-standard block title: .Input parameters
    • [MANUAL] modules/extend_orchestrator-in-rhdh/ref-compatibility-guide-for-orchestrator.adoc: Line 14: Non-standard block title: .Compatible Orchestrator and infrastructure versions
    • [MANUAL] modules/extend_orchestrator-in-rhdh/proc-enable-the-orchestrator-plugins-using-the-operator.adoc: Non-standard block title: .Example: Complete configuration of the Orchestrator plugin
    • [MANUAL] modules/extend_orchestrator-in-rhdh/proc-integrate-loki-logs-for-orchestrator-workflows.adoc: Line 21: Image(s) missing alt text in quotes
    • [MANUAL] modules/extend_orchestrator-in-rhdh/proc-create-a-serverless-workflow-project.adoc: Line 36: Image(s) missing alt text in quotes
    • [MANUAL] modules/extend_orchestrator-in-rhdh/proc-verify-the-deployment.adoc: Line 26: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-use-the-argo-cd-plugin.adoc: Line 19: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-use-the-jfrog-artifactory-plugin.adoc: Line 23: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-use-the-nexus-repository-manager-plugin.adoc: Line 21: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-use-the-tekton-plugin.adoc: Line 21: Image(s) missing alt text in quotes
    • [MANUAL] modules/shared/proc-use-the-topology-plugin.adoc: Line 20: Image(s) missing alt text in quotes
    • [MANUAL] modules/integrate_interacting-with-model-context-protocol-tools-for-rhdh/ref-accessing-rhdh-data-using-the-software-catalog-mcp-tools.adoc: Line 15: Non-standard block title: .Available tools in the software-catalog-mcp-tool plugin
    • [MANUAL] modules/integrate_interacting-with-model-context-protocol-tools-for-rhdh/ref-fetching-entities-using-fetch-catalog-entities.adoc: Line 16: Non-standard block title: .Parameters for the fetch-catalog-entities tool
    • [MANUAL] modules/integrate_interacting-with-model-context-protocol-tools-for-rhdh/ref-unregistering-entities-using-catalog-unregister-tool.adoc: Line 11: Non-standard block title: .Parameters for catalog-unregister-tool
    • [MANUAL] modules/integrate_interacting-with-model-context-protocol-tools-for-rhdh/ref-retrieving-software-template-metadata.adoc: Line 11: Non-standard block title: .Parameters for software-template-metadata-tool
    • [MANUAL] modules/integrate_interacting-with-model-context-protocol-tools-for-rhdh/ref-accessing-and-analyzing-documentation-using-the-techdocs-mcp-tools.adoc: Line 11: Non-standard block title: .TechDocs tools and parameters
    • [MANUAL] modules/integrate_interacting-with-model-context-protocol-tools-for-rhdh/ref-measuring-documentation-gaps-using-analyze-techdocs-coverage.adoc: Line 24: Non-standard block title: .Parameters for the analyze-techdocs-coverage TechDocs MCP tool
    • [MANUAL] modules/observability_audit-logs-in-rhdh/proc-forward-rhdh-audit-logs-to-splunk.adoc: Non-standard block title: .Example pipelines configuration
    • [MANUAL] modules/observability_audit-logs-in-rhdh/proc-forward-rhdh-audit-logs-to-splunk.adoc: Non-standard block title: .Example command to apply ClusterLogForwarder configuration
    • [MANUAL] modules/observability_audit-logs-in-rhdh/proc-forward-rhdh-audit-logs-to-splunk.adoc: Non-standard block title: .Example collector configuration
    • [MANUAL] modules/observability_audit-logs-in-rhdh/proc-forward-rhdh-audit-logs-to-splunk.adoc: Non-standard block title: .Example tuning configuration
    • [MANUAL] modules/observability_telemetry-data-collection-and-analysis/proc-disable-telemetry-data-collection-using-the-helm-chart.adoc: Line 36: Image(s) missing alt text in quotes
    • [MANUAL] modules/observability_telemetry-data-collection-and-analysis/proc-customize-segment-source-using-the-helm-chart.adoc: Line 25: Image(s) missing alt text in quotes
  • CQA RHIDP-208 Add missing images #4: Verify Module Templates
  • CQA RHIDP-131: Installing RHDH document #6: Verify Assemblies Follow Official Template (One User Story)
    • [MANUAL] assemblies/extend_installing-and-viewing-plugins-in-rhdh/assembly-front-end-plugin-wiring.adoc: Has 20 includes (consider splitting -- may cover multiple user stories, max 15)
  • CQA dummy commit to test asciidoctor gen for PRs #7: Verify TOC Depth (Max 3 Levels)
  • CQA Remove tracked build files #16: Verify Official Product Names
  • CQA RHIDP-452 Add attributes configuration and conscious language statement to Release Notes. #1: Vale AsciiDoc DITA Compliance
  • CQA RHIDP-449-2 Add theme mode content #12: Verify Grammar and Style (Vale)
  • CQA RHIDP-208-2 Red Hat Developer Hub Administration Guide - initial commit #17: Verify Legal Disclaimers for Preview Features
    • [MANUAL] modules/extend_dynamic-plugins-reference/ref-other-installable-plugins.adoc: Line 7: Mentions 'Technology Preview' but may not include official disclaimer snippet
    • [MANUAL] modules/shared/proc-navigate-the-plugin-marketplace-and-filter-plugins-using-badges.adoc: Line 20: Mentions 'Developer Preview' but may not include official disclaimer snippet
    • [MANUAL] modules/integrate_accelerating-ai-development-with-openshift-ai-connector-for-rhdh/proc-set-up-with.adoc: Line 282: Mentions 'Developer Preview' but may not include official disclaimer snippet
    • [MANUAL] modules/shared/con-about-and-llama-stack.adoc: Line 38: Mentions 'Developer Preview' but may not include official disclaimer snippet
    • [MANUAL] modules/shared/proc-install-and-configure.adoc: Line 17: Mentions 'Developer Preview' but may not include official disclaimer snippet
    • [MANUAL] modules/observability_adoption-insights-in-rhdh/proc-enable-the-adoption-insights-plugin.adoc: Line 7: Mentions 'Developer Preview' but may not include official disclaimer snippet
  • CQA RHIDP-208 Added dynamic plugin installation content #14: Verify No Broken Links
  • CQA RHIDP-442: RBAC feature documentation  #15: Check Redirects

Total: 19 checks | 16 passed | 3 with issues

To auto-fix what can be auto-fixed, run:

./build/scripts/cqa.sh --fix --all

To fix remaining issues, copy-paste this prompt to Claude:

Run ./build/scripts/cqa.sh --all, then for each failing CQA check, read the matching .claude/skills/cqa-*.md skill file and fix the [MANUAL] issues following the skill instructions.

Automated CQA check run on the entire repository

themr0c added a commit that referenced this pull request Apr 22, 2026
Co-authored-by: Priyanka Abel <pabel@redhat.com>
Co-authored-by: GitHub Actions <github-actions@github.com>
Co-authored-by: Lokananda Prabhu <102503482+lokanandaprabhu@users.noreply.github.com>
Co-authored-by: Lindsay Hite <lhite@redhat.com>
Co-authored-by: Judith Magak <124673476+jmagak@users.noreply.github.com>
Co-authored-by: Jessica He <jessicahe4741@gmail.com>
Co-authored-by: Kim Tsao <84398375+kim-tsao@users.noreply.github.com>
Co-authored-by: Nick Boldt <nboldt@redhat.com>
Co-authored-by: Fortune Ndlovu <ndlovufortune97@gmail.com>
Co-authored-by: Frank Kong <50030060+Zaperex@users.noreply.github.com>
Co-authored-by: Tomas Kral <tomas.kral@gmail.com>
Co-authored-by: Gerry-Forde <63045020+Gerry-Forde@users.noreply.github.com>
Co-authored-by: deerskindoll <jvrbkova@redhat.com>
Co-authored-by: Fabrice Flore-Thébault <ffloreth@redhat.com>
Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: rhdh-bot service account <rhdh-bot@redhat.com>
Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
Co-authored-by: Jana Vrbkova <jvrbkova@redhat.com>
Co-authored-by: Armel Soro <armel@rm3l.org>
Co-authored-by: Joseph Kim <joskim@redhat.com>
Co-authored-by: Kashish Mittal <113269381+04kash@users.noreply.github.com>
Co-authored-by: Gaurav Trivedi <90042568+gtrivedi88@users.noreply.github.com>
Co-authored-by: OpenShift Cherrypick Robot <openshift-cherrypick-robot@redhat.com>
Co-authored-by: Omar Al Jaljuli <o.jaljuli@mail.utoronto.ca>
Co-authored-by: Tim O'Keefe <tokeefe@redhat.com>
Fix typo in service account information section (#1697)
Fix build on release branches (#1703)
fix-asciidoc-warnings (#1700)
fix to unblock the builds (#1775)
Fix broken link (#1788)
Fixing some mistakes I made previously (#1787)
fixed errors (#1826)
fix arrays not merged by default (#1814)
fix: remove session-leaking parameters from documentation URLs (#1845)
fix build (#1858)
Fix image statements (#1859)
Fix image statements (#1861)
FIX-1 - RHDH 1.9 GA release edits (#1867)
Fixed scorecard broken link (#1873)
Fix concept module title per CQA 2.1 requirement #8:
fixes from IDs per CQA requirement #6
fixes from module IDs per CQA requirement #6.
Fixed module titles to comply with Requirement #8:
Fixed module and assembly titles to comply with Requirement #8:
fixing 404 error for clusters sharing 1 PVC (#1923)
fix for About Red Hat Developer Hub (#1934)
fix for Authentication (#1937)
fix for Adoption Insights (#1935)
fix for Audit Logs (#1936)
fixes for authorization title (#1938)
Fix CQA workflow PR comment failing on backticks (#1951)
fix remaining CQA errors (#1950)
Fix CQA PR comment rendering issues (#1952)
Fix CQA-5 and CQA-17 violations across repo (#1957)
Fix remaining CQA violations in 3 leftover files (#1958)
fix broken xrefs (#1959)
fix book-link/book-title attributes (#1961)
Fix broken link: Casbin link in Orchestrator docs [main] (#1973)
Fixing CQA 16 (#1985)
fixes from OCI artifact paths (#1963)
Fix broken link to dynamic-plugins.default.yaml and orphaned upgrade module (#1995)
Fix CQA workflow: base branch diff and comment duplication (#1999)
Fix PR preview comment duplication (#2002)
fix: improve CQA-17 disclaimer detection accuracy (#2008)
fix: CQA workflow fetch base branch from upstream repo (#2013)
Fix build orchestrator js-yaml dependency failure in CI (#2016)
Fix PR check status not reflecting build failure (#2017)
Fix shellcheck workflow creating duplicate PR comments (#2015)
Fix incorrect auth provider resolvers for GitHub (#2038)
Fix typo connexion -> connection in LDAP docs (#2039)
Resolve all existing CQA violations on main branch (#2052)
Fix empty CQA output in CI when checks fail (#2057)
Fix CQA/build-orchestrator infinite recursion (#2059)
fix copyrights/exec lines for the various scripts; add one for running the cqa so we can check that locally too (#2067)
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.

2 participants