Skip to content
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

Docs: Define Storybook's RFC process #23138

Merged
merged 9 commits into from
Jul 5, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -1,14 +1,11 @@
name: Feature request 💡
description: Suggest an idea for this project
title: '[Feature Request]:'
labels:
- needs triage
- feature request
body:
- type: textarea
id: problem
attributes:
label: Is your feature request related to a problem? Please describe
label: Is your feature request related to a problem? Please describe.
description: >-
A clear and concise description of the problem. E.g. I'm always
frustrated when [...]
Expand Down
10 changes: 8 additions & 2 deletions .github/ISSUE_TEMPLATE/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,16 @@ blank_issues_enabled: false
contact_links:
- name: View documentation 📚
url: https://storybook.js.org/docs/
about: Check out the official docs for answers to common questions
about: Check out the official docs for answers to common questions.
- name: Feature Requests 💡
url: https://github.com/storybookjs/storybook/discussions/new?category=ideas
about: Suggest a feature idea for this project.
- name: Open an RFC 🦄
url: https://github.com/storybookjs/storybook/discussions/new?category=rfc
about: Do you want to propose a more involved change to Storybook? Open an RFC (Request for Comments) to start a discussion.
- name: Questions & discussions 🤔
url: https://github.com/storybookjs/storybook/discussions
about: Ask questions, request features & discuss RFCs
about: Ask questions, show off your Storybook, etc.
- name: Community Discord 💬
url: https://discord.gg/storybook
about: Community discussions, interactive support, contributor help
55 changes: 55 additions & 0 deletions docs/contribute/RFC.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,55 @@
---
title: 'RFC process'
---

The RFC (Request for Comment) process is intended to provide a consistent and controlled path for new features to enter the project. It helps ensure that new features are well-designed, well-implemented, and well-tested, and they do not conflict with the project's overall direction and scope.

## Goal

Many changes, such as bug fixes and documentation improvements, can be implemented and reviewed via the normal GitHub pull request workflow. Some changes, however, are considered “substantial”, and we ask that these undergo a design process, solicit community input, and reach a consensus among the Storybook core team.

The purpose of the RFC (Request for Comment) process is to:

- Provide a transparent system for proposing new feature ideas.
- Establish a reliable and well-regulated process for introducing new features into the project.
- Provide a way for the community to participate in developing new features.

### “Feature Request” vs. “RFC”

A _feature request_ is a straightforward and relatively informal way for Storybook users to suggest a new feature or enhancement to the project. While feature requests can provide valuable insights and ideas, they typically do not involve an in-depth design process or require consensus among the core team. Feature requests are usually open to discussion and may or may not be implemented based on factors like popularity, feasibility, and alignment with the project's goals.

On the other hand, an _RFC_ is a more formalized and structured process for proposing substantial changes or additions to the project. It involves following a defined set of steps to ensure that the proposed feature or modification receives proper consideration, design, and feedback. RFCs are typically used for changes that significantly impact the project, such as introducing new API functionality, removing existing features, or establishing new usage conventions. The RFC process aims to foster discussions, gather feedback from a wider audience, and reach consensus among the core team before integrating the proposed change into the project. Accepted RFCs are more likely to be implemented than regular feature requests.

## The RFC lifecycle

### 1. `Status: Proposed`

Open a new GitHub discussion in the [“RFC” category](https://github.com/storybookjs/storybook/discussions/new?category=rfc). Fill out the form as instructed.

_Details matter_: RFCs that do not present convincing motivation, demonstrate a lack of understanding of the design's impact, or are disingenuous about the drawbacks or alternatives tend to be poorly received.

### 2. `Status: In review`

RFCs tend to remain in this stage for a while, giving the community and core team members time to weigh in. During this period, the author of an RFC should be prepared to revise the proposal, integrate feedback, and build consensus. RFCs that have broad support are much more likely to make progress than those that don't receive any comments.

Every Monday at 11 a.m. (EST), the Storybook core team conducts a weekly triage meeting to review open RFCs as part of the meeting's agenda. The meeting is held in the [Storybook Discord's Watercooler channel](https://discord.com/channels/486522875931656193/486522876388704260). We invite the RFC author(s) and interested members of the community to participate and engage in a more detailed discussion of the RFC. If a core team member deems it necessary, they will be assigned as the "champion" of the RFC. The champion will collaborate with the RFC author and assist them throughout the RFC process.

### 3. `Status: accepted/rejected`

Eventually, the team will decide whether the RFC is a candidate for inclusion in Storybook. On the other hand, an RFC may be rejected by the team after a public discussion has settled and comments have been made summarizing the rationale for rejection.

## Implementing an accepted RFC

The author of an RFC is not obligated to implement it. Of course, the RFC author (like any other developer) is welcome to post an implementation for review after the RFC has been accepted. However, note that the “accepted” status does not indicate priority nor whether it’s being actively worked on.

If you are interested in implementing an "active" RFC, but cannot determine if someone else is already working on it, feel free to ask (e.g., by leaving a comment on the associated issue).

This RFC process took heavy inspiration from the RFC processes from [Rust](https://github.com/rust-lang/rfcs) and [Gatsby](https://www.gatsbyjs.com/contributing/rfc-process/).

## Learn more about contributing to Storybook

- RFC process for authoring feature requests
- [Code](./code.md) for features and bug fixes
- [Frameworks](./framework.md) to get started with a new framework
- [Documentation](./documentation-updates.md) for documentation improvements, typos, and clarifications
- [Examples](./new-snippets.md) for new snippets and examples
13 changes: 6 additions & 7 deletions docs/contribute/code.md
Original file line number Diff line number Diff line change
Expand Up @@ -254,11 +254,10 @@ It's troublesome to know which packages you'll change ahead of time, and watchin

</details>

## Other ways to contribute
## Learn more about contributing to Storybook

Learn about other ways you can contribute to Storybook.

- [**Overview**](./how-to-contribute.md): General guidance
- [**Docs**](./documentation-updates.md): Typos, clarifications
- [**Addons**](./../addons/introduction.md): Build an addon and share it with the community
- [**Frameworks**](./framework.md): Integrate Storybook with your favorite library
- [RFC process](./RFC.md) for authoring feature requests
- Code for features and bug fixes
- [Frameworks](./framework.md) to get started with a new framework
- [Documentation](./documentation-updates.md) for documentation improvements, typos, and clarifications
- [Examples](./new-snippets.md) for new snippets and examples
13 changes: 6 additions & 7 deletions docs/contribute/documentation-updates.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,10 @@ Scroll down to the bottom of the document page on GitHub and describe what you c

In the Storybook repository, create a pull request that describes changes and includes additional context that would help maintainers review. Once you submit the PR, a maintainer will guide you through the triage and merge process.

## Other ways to contribute
## Learn more about contributing to Storybook

Learn about other ways you can contribute to Storybook.

- [**Overview**](./how-to-contribute.md): General guidance
- [**Code**](./code.md): Features, bug fixes, dependency updates
- [**Addons**](./../addons/introduction.md): Build an addon and share it with the community
- [**Frameworks**](./framework.md): Integrate Storybook with your favorite library
- [RFC process](./RFC.md) for authoring feature requests
- [Code](./code.md) for features and bug fixes
- [Frameworks](./framework.md) to get started with a new framework
- Documentation for documentation improvements, typos, and clarifications
- [Examples](./new-snippets.md) for new snippets and examples
13 changes: 6 additions & 7 deletions docs/contribute/framework.md
Original file line number Diff line number Diff line change
Expand Up @@ -144,11 +144,10 @@ Test it in a fresh project using a Storybook set up as close as possible to your

Once it's fully tested and released, please let us know about your framework by either announcing it in the `#showcase` channel of the [Storybook Discord](https://discord.gg/storybook) or tweeting it and mentioning `@storybookjs`. It's our hope that well-made community frameworks can eventually move into the Storybook codebase and be considered "officially" supported.

## Other ways to contribute
## Learn more about contributing to Storybook

Learn about other ways you can contribute to Storybook.

- [**Contribution overview**](./how-to-contribute.md): General guidance
- [**Code**](./code.md): Features, bug fixes, dependency updates
- [**Docs**](./documentation-updates.md): Typos, clarifications
- [**Addons**](./../addons/introduction.md): Build an addon and share it with the community
- [RFC process](./RFC.md) for authoring feature requests
- [Code](./code.md) for features and bug fixes
- Frameworks to get started with a new framework
- [Documentation](./documentation-updates.md) for documentation improvements, typos, and clarifications
- [Examples](./new-snippets.md) for new snippets and examples
12 changes: 7 additions & 5 deletions docs/contribute/how-to-contribute.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,14 +6,16 @@ Storybook is a community-oriented open source project that welcomes contribution

## Contributor covenant

In the interest of fostering an open and welcoming environment, we as contributors and maintainers pledge to making participation in our project and our community a harassment-free experience for everyone, regardless of age, body size, disability, ethnicity, gender identity and expression, level of experience, nationality, personal appearance, race, religion, or sexual identity and orientation.[Continue reading our contributor covenant »](https://github.com/storybookjs/storybook/blob/next/CODE_OF_CONDUCT.md)
In the interest of fostering an open and welcoming environment, we as contributors and maintainers pledge to making participation in our project and our community a harassment-free experience for everyone, regardless of age, body size, disability, ethnicity, gender identity and expression, level of experience, nationality, personal appearance, race, religion, or sexual identity and orientation. [Continue reading our contributor covenant »](https://github.com/storybookjs/storybook/blob/next/CODE_OF_CONDUCT.md)

## Ways to contribute

- [**Code**](./code.md): Features, bug fixes, dependency updates
- [**Docs**](./documentation-updates.md): Typos, clarifications
- [**Addons**](./../addons/introduction.md): Build an addon and share it with the community
- [**Frameworks**](./framework.md): Integrate Storybook with your favorite library
- [**RFC process**](./RFC.md) for authoring feature requests
- [**Code**](./code.md) for features and bug fixes
- [**Frameworks**](./framework.md) to get started with a new framework
- [**Documentation**](./documentation-updates.md) for documentation improvements, typos, and clarifications
- [**Examples**](./new-snippets.md) for new snippets and examples
- [**Addons**](./../addons/introduction.md) for new addons

## Not sure how to get started?

Expand Down
8 changes: 8 additions & 0 deletions docs/contribute/new-snippets.md
Original file line number Diff line number Diff line change
Expand Up @@ -126,3 +126,11 @@ Go through the documentation and check your work.
## Submit your contribution

Finally, commit, push and open a pull request in the Storybook monorepo. Add a clear description of the work you've done, and one of the maintainers will guide you through the merge process.

## Learn more about contributing to Storybook

- [RFC process](./RFC.md) for authoring feature requests
- [Code](./code.md) for features and bug fixes
- [Frameworks](./framework.md) to get started with a new framework
- [Documentation](./documentation-updates.md) for documentation improvements, typos, and clarifications
- [Examples](./new-snippets.md) for new snippets and examples
5 changes: 5 additions & 0 deletions docs/toc.js
Original file line number Diff line number Diff line change
Expand Up @@ -680,6 +680,11 @@ module.exports = {
title: 'How to',
type: 'link',
},
{
pathSegment: 'RFC',
title: 'RFC Process',
type: 'link',
},
{
pathSegment: 'code',
title: 'Code',
Expand Down