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

Updates to stable criteria #9

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Updates to stable criteria #9

wants to merge 1 commit into from

Conversation

yaili
Copy link
Member

@yaili yaili commented Dec 14, 2021

Done:

  • Added Figma component to alpha and beta

Todo:

  • Review accessibility criteria
  • Review design review criteria
  • Review design guidelines criteria

Done:
- Added Figma component to alpha and stable

Todo:
- Review accessibility criteria
- Review design review criteria
- Review design guidelines criteria
@vercel
Copy link

vercel bot commented Dec 14, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/monalisa/primer-contribute/9XtbmStTD8mVHmUDXqfgTTgdkPdS
✅ Preview: https://primer-contribute-git-update-stable-criteria-monalisa.vercel.app

@@ -20,6 +20,7 @@ The component is ready for preliminary usage, with breaking changes expected.
- Basic documentation exists that includes example usage of the component.

Choose a reason for hiding this comment

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

It's been hard for me to know whether my components meet this criteria because "basic documentation" is so vague.

What criteria do we have for "basic"?

What do we mean by "documentation"? Do we mean design guidelines? Engineering docs?

cc @colebemis - we talked about this a few weeks ago, and you might have some thoughts on this criteria.

@@ -20,6 +20,7 @@ The component is ready for preliminary usage, with breaking changes expected.
- Basic documentation exists that includes example usage of the component.
- Primary use cases tested and reviewed.
- The component has 100% test coverage.
- The default component is available in the Primer Web library in Figma.

Choose a reason for hiding this comment

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

I don't understand what we mean by "default component". I'd recommend we don't make Figma implementation an Alpha requirement. I think it can wait until Beta.


## Stable

The component is significantly mature and usage is strongly encouraged, with long-term support expected.

- The API remains stable, with no breaking changes for at least one month. Feedback on API usability has been sought from developers using the component and production use cases have been reviewed for correctness.
- Documentation exists for the remaining implementation, usage, and design guidelines. Primer React and/or Primer ViewComponents documentation includes all props and variations, accessibility guidelines (including common misuses), and common scenarios. Design documentation is added to Interface Guidelines. Figma components are available in the Primer Web library.
- Documentation exists for the remaining implementation, usage, and design guidelines. Primer React and/or Primer ViewComponents documentation includes all props and variations, accessibility guidelines (including common misuses), and common scenarios. Design documentation is added to Interface Guidelines.

Choose a reason for hiding this comment

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

Primer React and/or Primer ViewComponents documentation includes all props and variations

"Stable" seems a little late for this. If product teams using these components when they're in "Alpha" or "Beta", they should at least have props and variations documented.

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