Skip to content

Fix plugin validation workflow reports and scope - #82

Merged
wu-clan merged 2 commits into
fastapi-practices:masterfrom
wu-clan:fix/plugin-validation-workflow
Oct 29, 2026
Merged

wu-clan merged 2 commits into
fastapi-practices:masterfrom
wu-clan:fix/plugin-validation-workflow

Conversation

@wu-clan

@wu-clan wu-clan commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Render validation reports directly as Markdown instead of wrapping them in a collapsed details block
  • Sanitize PR-controlled report content before posting it to the pull request
  • Validate only plugin submodules changed by the pull request
  • Avoid falling back to a full validation when no plugin submodule changes

Validation

  • actionlint .github/workflows/*.yml
  • YAML parsing
  • Bash and JavaScript syntax checks
  • GitHub Markdown API render check: headings and tables render correctly, with no <details> output
  • git diff --check

Note

Comment Plugin Validation is triggered by workflow_run, so GitHub executes the workflow from the default branch. The formatting update in this PR will take effect for subsequent validation runs after it is merged.

@github-actions

github-actions Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Plugin TOML Validation Report

Status: ✅ No plugin submodules changed

No plugin TOML validation was required for this pull request

View workflow logs

@wu-clan
wu-clan merged commit 86fd618 into fastapi-practices:master Oct 29, 2026
1 check passed
@wu-clan
wu-clan deleted the fix/plugin-validation-workflow branch October 29, 2026 08:02
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.

1 participant