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

Conversation

vanessayuenn
Copy link
Contributor

What I did

  • Direct feature request to opening a GitHub discussion instead of an issue (cc @shilman)
  • Add a callout to opening RFCs in issues/new
  • Formalize our "RFC process" by documenting it.

Outstanding tasks

  • Find a home for the RFC doc. I just threw it under contributing for now, but @jonniebigodes @kylegach if you have thoughts on where this should live long term, please let me know.
  • Make sure the document is linked from appropriate places within the docs site

Maintainers

  • If this PR should be tested against many or all sandboxes,
    make sure to add the ci:merged or ci:daily GH label to it.
  • Make sure this PR contains one of the labels below.

["cleanup", "BREAKING CHANGE", "feature request", "bug", "documentation", "maintenance", "dependencies", "other"]

@vanessayuenn vanessayuenn requested review from a team and removed request for a team June 19, 2023 16:02
@vanessayuenn vanessayuenn changed the title Define Storybook's RFC process Docs: Define Storybook's RFC process Jun 19, 2023
@vanessayuenn vanessayuenn marked this pull request as ready for review June 19, 2023 16:06
Copy link
Contributor

@JReinhold JReinhold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great!

docs/contribute/RFC.md Outdated Show resolved Hide resolved
docs/contribute/RFC.md Outdated Show resolved Hide resolved
@jonniebigodes jonniebigodes self-assigned this Jun 19, 2023
docs/contribute/RFC.md Outdated Show resolved Hide resolved
docs/contribute/RFC.md Outdated Show resolved Hide resolved
docs/contribute/RFC.md Outdated Show resolved Hide resolved
Copy link
Contributor

@kylegach kylegach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, Vanessa! The content looks great to me, but I left a comment about the file's location that must be addressed.

Copy link
Contributor

@jonniebigodes jonniebigodes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vanessayuenn left one item for you to look into that I'm more than glad to follow up with. Let me know and we'll go from there.

docs/contribute/RFC.md Outdated Show resolved Hide resolved
vanessayuenn and others added 2 commits June 21, 2023 18:16
Co-authored-by: Valentin Palkovic <valentin@chromatic.com>
Co-authored-by: Kyle Gach <kyle.gach@gmail.com>
@kylegach kylegach dismissed their stale review June 21, 2023 16:22

I'll let João manage it from here, as he has an identical understanding (and then some).

@jonniebigodes
Copy link
Contributor

Will merge once the checklist clears. Once again thanks for putting this together @vanessayuenn. 🙏

@jonniebigodes jonniebigodes merged commit 992efdf into next Jul 5, 2023
@jonniebigodes jonniebigodes deleted the vy/feature-requests-as-discussions branch July 5, 2023 17:06
@github-actions github-actions bot mentioned this pull request Jul 5, 2023
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants