Skip to content

[RHDHBUGS-3010]: Fix shellcheck workflow on deleted .sh files#2102

Merged
themr0c merged 1 commit into
redhat-developer:mainfrom
themr0c:fix/shellcheck-deleted-files
Apr 24, 2026
Merged

[RHDHBUGS-3010]: Fix shellcheck workflow on deleted .sh files#2102
themr0c merged 1 commit into
redhat-developer:mainfrom
themr0c:fix/shellcheck-deleted-files

Conversation

@themr0c

@themr0c themr0c commented Apr 24, 2026

Copy link
Copy Markdown
Member

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

Version(s): All
Issue: https://redhat.atlassian.net/browse/RHDHBUGS-3010
Preview: N/A

Summary

  • Filter out deleted .sh files before running shellcheck, preventing reviewdog: parse error: EOF when a PR only deletes shell scripts

Merge this before /pull/2100 and /pull/2101 so the shellcheck workflow runs correctly on those PRs.

Cherry-pick to release-1.9 after merge.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

Filter out deleted shell scripts before running shellcheck.
When a PR only deletes .sh files, reviewdog received empty
checkstyle input and failed with "parse error: EOF".

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@themr0c themr0c merged commit adf1293 into redhat-developer:main Apr 24, 2026
3 of 4 checks passed
@sonarqubecloud

Copy link
Copy Markdown

@rhdh-bot

Copy link
Copy Markdown
Collaborator

PR Build Results

Build failed -- 34/34 titles | 114s
Preview: https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-2102/

View full logs


Content Quality Assessment

CQA Report

  • CQA-00a: Orphaned modules
  • CQA-00b: Directory structure
  • CQA-01: Vale AsciiDoc DITA compliance
  • CQA-02: Verify assembly structure
  • CQA-03: Verify content type metadata
  • CQA-04: Verify module templates
  • CQA-05: Verify required modular elements
  • CQA-06: Verify assemblies follow official template (one user story)
  • CQA-07: Verify TOC depth (max 3 levels)
  • CQA-08: Verify short description content quality
  • CQA-09: Verify short description format
  • CQA-10: Verify titles are brief, complete, and descriptive
  • CQA-11: Verify procedure prerequisites
  • CQA-12: Verify grammar and style (Vale)
  • CQA-13: Verify content matches declared type
  • CQA-14: Verify no broken links
  • CQA-15: Check redirects
  • CQA-16: Verify official product names
  • CQA-17: Verify legal disclaimers for preview features

Summary

Checks: 19 total, 19 pass, 0 fail

19 checks: 19 pass, 0 fail

Run node build/scripts/cqa/index.js --all --fix locally to review and auto-fix issues.


Updated 2026-04-24 14:10:13 UTC

@themr0c themr0c deleted the fix/shellcheck-deleted-files branch April 29, 2026 11:42
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