Skip to content

qa: tighten go-live gates and add readiness validation artifacts#192

Merged
accesswatch merged 5 commits intomainfrom
qa/readiness-gates-20260508
May 8, 2026
Merged

qa: tighten go-live gates and add readiness validation artifacts#192
accesswatch merged 5 commits intomainfrom
qa/readiness-gates-20260508

Conversation

@accesswatch
Copy link
Copy Markdown
Collaborator

Summary\n- Tighten go-live release gates and explicitly align with the E2E runbook\n- Add local non-podcast readiness test coverage and documented evidence\n- Add student repository restore/recovery script\n- Add qa-bundle snapshot artifacts for classroom and readiness validation\n\n## Validation\n- npm run test:automation (99 passed)\n\n## Notes\n- Direct pushes to main are blocked by repository rules; this PR is required for merge.

GitHub Workshop Bot added 5 commits May 8, 2026 11:23
- Restructure docs and classroom content
- Add .gitattributes to enforce LF line endings across all text files
- Remove deprecated podcast bundles, scripts, and transcripts
- Remove old HTML file mappings replaced by updated structure
- Update admin, docs, html, and learning-room content
Copilot AI review requested due to automatic review settings May 8, 2026 22:26
@accesswatch accesswatch requested a review from taylorarndt as a code owner May 8, 2026 22:26
Copy link
Copy Markdown

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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Peer Review Assigned

Hi @accesswatch! Your PR has been automatically paired with @taylorarndt for peer review.

For @taylorarndt:

This is a great opportunity to practice code review skills! Here's what to look for:

Content Quality:

  • Does the change accomplish what the issue describes?
  • Is the writing clear and helpful?
  • Are there any typos or grammar issues?

Accessibility:

  • Proper heading hierarchy (H1 → H2 → H3, no skips)?
  • Descriptive link text (not "click here")?
  • Alt text on images?
  • [TODO] markers removed?

Documentation:

  • Code blocks are properly formatted?
  • Tables have headers?
  • References/links work correctly?

Review Guidelines:

  • Be kind and constructive
  • Suggest improvements, don't just point out problems
  • Ask questions if something is unclear
  • Approve when ready or request changes with explanation

Resources:


Pairing by Learning Room Grouping Engine

@accesswatch accesswatch merged commit faa4f33 into main May 8, 2026
3 checks passed
@accesswatch accesswatch deleted the qa/readiness-gates-20260508 branch May 8, 2026 22:31
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