Skip to content

[schema-coverage] feat: add schema coverage demo for resources field#36203

Merged
pelikhan merged 1 commit into
mainfrom
schema-coverage-resources-c1ef657d89e3f5fa
Jun 1, 2026
Merged

[schema-coverage] feat: add schema coverage demo for resources field#36203
pelikhan merged 1 commit into
mainfrom
schema-coverage-resources-c1ef657d89e3f5fa

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Jun 1, 2026

Summary

Adds a new schema coverage demo file for the resources frontmatter field.

What changed

File Change
schema-demos/schema-demo-resources.md ➕ Added

Why

Improves schema field coverage by adding a dedicated demo workflow document for the resources frontmatter field, consistent with the existing suite of schema demo files.

Details

  • schema-demos/schema-demo-resources.md — New demo workflow file that documents and demonstrates the resources frontmatter schema field for schema coverage purposes. No logic changes; documentation/demo only.

Risk

🟢 Low — Documentation/demo file only. No functional code changes. No breaking changes.

Generated by PR Description Updater for issue #36203 · sonnet46 630.5K ·

@pelikhan pelikhan marked this pull request as ready for review June 1, 2026 11:21
Copilot AI review requested due to automatic review settings June 1, 2026 11:21
@pelikhan pelikhan merged commit f8d4a3c into main Jun 1, 2026
2 checks passed
@pelikhan pelikhan deleted the schema-coverage-resources-c1ef657d89e3f5fa branch June 1, 2026 11:21
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions Bot commented Jun 1, 2026

Design Decision Gate 🏗️ completed the design decision gate check.

No ADR enforcement needed: PR #36203 does not have the 'implementation' label and has 0 new lines of code in business logic directories (threshold is 100).

@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions Bot commented Jun 1, 2026

🧪 Test Quality Sentinel completed test quality analysis.

No test files were added or modified in this PR. The only changed file is schema-demos/schema-demo-resources.md (31 additions, 0 deletions). Test Quality Sentinel skipped.

@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions Bot commented Jun 1, 2026

PR Code Quality Reviewer completed the code quality review.

@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions Bot commented Jun 1, 2026

⚠️ Security scanning failed for Matt Pocock Skills Reviewer. Review the logs for details.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new schema coverage demo workflow to exercise the resources frontmatter field so the schema feature coverage checker can find at least one usage.

Changes:

  • Introduces schema-demos/schema-demo-resources.md with minimal frontmatter and a resources: entry.
  • Documents the purpose of resources and includes a noop safe-output call for coverage-only execution.
Show a summary per file
File Description
schema-demos/schema-demo-resources.md New minimal schema demo intended to cover the resources frontmatter field.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

contents: read
engine: codex
resources:
- ../.github/workflows/ci.yml
Copy link
Copy Markdown
Contributor Author

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

No blocking issues. Single markdown demo file adding schema coverage for the resources field — straightforward and correct.

🔎 Code quality review by PR Code Quality Reviewer · sonnet46 430.9K

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants