RFC review process
This document explains RFC review process for @fluentui/react-components
The goal of this process is to ensure that RFCs and specs have the required attention and that authors receive meaningful feedback.
Currently we have the following teams that work on the library (each team has a CREW LEAD):
- @microsoft/cxe-coastal
- @microsoft/cxe-prg
- @microsoft/cxe-red
- @microsoft/teams-prg
RFCs and specs that don't have REVIEWERS are announced at the beginning of offline Tech Syncs that happen weekly on Wednesdays. The goal of a CREW LEAD is to assign a REVIEWER from their team and provide an estimation for an initial review. A TEAM may also skip a review, but that decision should be recorded on a PR through a comment. A CREW LEAD should comment on a PR by Friday EOD.
Examples of a comment:
@gh-handle will review on behalf @microsoft/TEAM by 30 Apr.
Our team will not review this PR, we kindly request that you use your best judgement in determining the next steps for this pull request 🐱
- A comment from a CREW LEAD provides clarity on when and who will participate in reviews
- RFCs/specs can still be reviewed by anyone
- Estimation date is only for providing an initial review, this does not mean that RFC/spec will be merged that day
- FAQ - Fabric and Stardust to Fluent UI
-
@fluentui/react
Version 9 -
@fluentui/react
Version 8 - Contributing to the
7.0
branch - How to apply themes (version 7/8)
- Planning and development process (for work by the core team)
- Conducting meetings Style guide
- Keeping up with review requests
- RFC review process
- Setup (configuring your environment)
- Fluent UI React version 7/8
- CLA
- Overview
- Repo structure
- Development process
- Contributing to previous versions
- API Extractor
- Build command changes made in early 2020
- Component implementation guide
- Creating a component
- Implementation Best Practices
- Theming
- Documenting
- Styling (old approach)
- Overview
- Testing with Jest
- E2E testing (Cypress)
- Visual testing (Screener)
- Accessibility review checklist