Review Process
Pull Request Review Workflow
Review SLAs
| PR Type | Target Review Time | Priority |
|---|---|---|
| Critical fixes (broken links, security) | 24 hours | High |
| Content updates (typos, clarifications) | 48 hours | Medium |
| New content (pages, guides) | 72 hours | Medium |
| Major restructures (IA changes) | 1 week | Low |
Review Rubric
Each PR is evaluated against:| Criterion | Description | Must Fix / Optional |
|---|---|---|
| Clarity & Comprehension | Is the content clear and easy to understand? | Must fix if unclear |
| Technical Accuracy | Are the facts correct and up-to-date? | Must fix if inaccurate |
| Completeness | Does it cover the topic adequately? | Optional improvements |
| UX & IA | Does it fit well in the information architecture? | Must fix if breaks IA |
| Strategic Alignment | Does it align with Livepeer’s goals? | Optional consideration |
| Style Consistency | Does it follow style guides? | Must fix if inconsistent |
Ownership
Section Owners
Documentation sections have designated owners who are responsible for reviewing changes and maintaining quality:| Section | Path | Owners |
|---|---|---|
| AI & Gateways | v2/developers/ai-inference-on-livepeer/, v2/pages/04_gateways/ | @rickstaa |
| Developers | v2/developers/ | @livepeer/studio-team, @DeveloperAlly |
| About | v2/pages/01_about/ | @livepeer/studio-team, @DeveloperAlly |
| Orchestrators | v2/orchestrators/ | @livepeer/studio-team |
| Delegators | v2/pages/06_delegators/ | @livepeer/studio-team |
| Resources | v2/resources/ | @livepeer/studio-team, @DeveloperAlly |
| Help | v2/pages/08_help/ | @livepeer/studio-team, @DeveloperAlly |
| Home | v2/home/ | @livepeer/studio-team, @DeveloperAlly |
| Products | v2/pages/010_products/ | @livepeer/studio-team |
.github/CODEOWNERS
Owner Responsibilities
Section owners are responsible for:- ✅ Reviewing PRs in their section
- ✅ Maintaining technical accuracy
- ✅ Ensuring style consistency
- ✅ Updating content as products evolve
- ✅ Responding to questions about their section
Becoming an Owner
Interested in becoming a section owner?- Make consistent, high-quality contributions to a section
- Demonstrate expertise in the subject matter
- Express interest to current maintainers
- Current owners will evaluate and may invite you
Ticketing System
Reporting Issues
Use GitHub Issues to report problems, suggest improvements, or ask questions: Issue Types:| Type | When to Use | Template |
|---|---|---|
| Bug Report | Broken links, incorrect information, formatting issues | Bug Report Template |
| Feature Request | New content, improvements, enhancements | Feature Request Template |
| Question | Clarifications, how-to questions | Question Template |
| Documentation Request | Missing documentation, unclear explanations | Use Feature Request template |
Issue Labels
We use labels to categorise issues, classify severity/impact, and set maintainer scheduling priority: Classification (severity/impact):classification: urgent- Immediate triage needed (release-blocking, unsafe guidance, or widespread breakage)classification: high- Major blocker or major user impactclassification: moderate- Noticeable friction/confusion with limited scope or workaroundclassification: minor- Cosmetic or low-risk localised issue (links, style, small wording fixes)
classification:*= severity/impact of the issue as reportedpriority:*= maintainer scheduling/sequence decision (may differ due to roadmap, staffing, or deadlines)
- Docs/page issue: broken link or styling inconsistency on one page ->
classification: minor - Tooling/CI issue: default-branch required CI workflow failing ->
classification: urgent
priority: critical- Security issues, broken critical pathspriority: high- Important content gaps, user blockerspriority: medium- Standard improvementspriority: low- Nice-to-have enhancements
type: bug- Something is brokentype: enhancement- Improvement or new featuretype: documentation- Documentation-relatedtype: question- Question or clarification needed
area: ai- AI/Gateway documentationarea: developers- Developer documentationarea: orchestrators- Orchestrator documentationarea: gateways- Gateway documentationarea: about- About sectionarea: resources- Resources section
status: needs-triage- Needs initial reviewstatus: in-progress- Work in progressstatus: blocked- Blocked on somethinggood first issue- Good for new contributors
Issue SLAs
| Issue Type | Target Response | Target Resolution |
|---|---|---|
| Priority: critical | 24 hours | 1 week |
| Priority: high | 48 hours | 2 weeks |
| Priority: medium | 1 week | 1 month |
| Priority: low | 2 weeks | As capacity allows |
Triage Process
- Initial Triage - A maintainer reviews and labels the issue
- Assignment - Issue is assigned to a section owner or contributor
- Work - Contributor works on the issue
- Review - PR is reviewed and merged
- Close - Issue is closed when resolved
Reviewers
Core Reviewers
| Role | Responsibilities | Contact |
|---|---|---|
| Technical Director | Overall documentation strategy, unified voice | Via GitHub |
| Section Owners | Review PRs in their section | See CODEOWNERS |
| Subject Matter Experts | Technical accuracy review | @rickstaa (AI), others TBD |
| Community Maintainers | Help review community contributions | @DeveloperAlly |
Review by Category
For major changes, we may request review from:- Site-wide changes - All maintainers
- Home/About - @livepeer/studio-team, @DeveloperAlly
- Developers - @livepeer/studio-team, @DeveloperAlly
- Gateways - @rickstaa
- Orchestrators - @livepeer/studio-team
- Resources - @livepeer/studio-team, @DeveloperAlly
Community Reviewers
Active contributors may be invited to become reviewers. Criteria:- Consistent, high-quality contributions
- Good understanding of documentation standards
- Willingness to help review PRs
Versioning and Deprecation
Versioning Policy
- Current version:
v2(default) - Legacy version:
v1(maintained for reference) - Versioning model: URL-based (
/v2/...,/v1/...)
Deprecation Process
When deprecating content:- Mark as deprecated - Add deprecation notice to page
- Create redirect - Add redirect in
docs.json - Update references - Update all links pointing to deprecated content
- Archive - Move to
v1/or archive location - Document - Update changelog with deprecation notice
Changelog
All major changes are documented in the Changelog.Quarterly Review Process
Every quarter, we review:- ✅ Content accuracy and freshness
- ✅ Broken links and references
- ✅ User feedback and common questions
- ✅ Content gaps and improvements
- ✅ Style guide compliance
- ✅ Information architecture effectiveness
Quarterly Review Checklist
Use this checklist for quarterly documentation reviews:Content Quality
- All pages reviewed for accuracy
- Outdated information updated
- Broken links fixed
- Examples tested and verified
- Code snippets updated if APIs changed
User Experience
- User feedback reviewed and addressed
- Common questions documented
- Navigation structure optimised
- Search functionality working well
- Mobile experience verified
Technical
- Build process working
- CI/CD checks passing
- Performance optimised
- SEO metadata current
- Analytics reviewed
Governance
- CODEOWNERS up to date
- Review process documented
- SLAs being met
- Contributor onboarding smooth
- Style guides current
Questions?
- GitHub Issues - Open an issue for governance questions
- Discord - Join #docs for discussion
- Email - Contact maintainers directly for sensitive matters