-
Notifications
You must be signed in to change notification settings - Fork 28
Add daily documentation testing workflow with beginner perspective #4088
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds an automated documentation quality assurance workflow that tests the documentation from a beginner's perspective. The workflow uses the Codex engine with Playwright to navigate the documentation site, identify confusing or broken steps, and report findings in GitHub discussions.
Key Changes:
- Daily automated testing of documentation from a novice user perspective
- Browser automation using Playwright to navigate key documentation pages
- Comprehensive reporting via GitHub discussions with screenshots and categorized findings
Reviewed Changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.github/workflows/docs-noob-tester.md |
Main workflow definition with Codex engine, Playwright tools, and safe-outputs for discussion creation |
.github/workflows/docs-noob-tester.lock.yml |
Compiled GitHub Actions workflow (auto-generated, 3967 lines) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| timeout-minutes: 30 | ||
| tools: | ||
| playwright: | ||
| version: "v1.56.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot do not pin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed version pinning. Playwright will now use the latest version. (4bc0611)
…mmand, add defaults to network Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
| workflow_dispatch: | ||
| permissions: | ||
| contents: read | ||
| discussions: write |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot remove, not needed thanks to safe outputs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed discussions: write permission. Safe-outputs handles write permissions automatically. (378c948)
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
| - "localhost" | ||
| - "127.0.0.1" | ||
| bash: | ||
| - "npm install*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot allow * in bash
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Simplified bash configuration to allow all commands with "*" wildcard. (ca08fc1)
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot use copilot agent engine |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Create Documentation Testing Agentic Workflow ✅
Successfully created a "noob" agentic workflow that tests the documentation site and reports confusing or broken steps.
Completed Tasks
.github/workflows/docs-noob-tester.md./gh-aw compilegh-aw statusRecent Changes (PR Feedback)
npm run dev*from bash commands (not needed)defaultsto network allowed list (follows convention)discussions: writepermission (safe-outputs handles permissions automatically)"*"(allows all commands)Implementation Summary
Workflow File:
.github/workflows/docs-noob-tester.mdCompiled File:
.github/workflows/docs-noob-tester.lock.ymlKey Configuration:
What the Workflow Does:
Testing:
workflow_dispatchOriginal prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.