Add quality scale #92
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
||
| # Platinum 🎖 | ||
|
|
||
| Best of the best. The integration is completely async, meaning it's super fast. Integrations that reach expert level will require the code owner approval for each PR. |
There was a problem hiding this comment.
Integrations that reach platinum level will require approval by the code owner for each new PR.
There was a problem hiding this comment.
Should this also require some SLA from the owner?
OverloadUT
reviewed
Sep 25, 2018
OverloadUT
reviewed
Sep 25, 2018
7027654 to
4cc861f
Compare
|
Addressed comments. Also updated the emoji for the platinum level 😉 |
| sidebar_label: "Introduction" | ||
| --- | ||
|
|
||
| The Integration Quality Scale scores each integration based on the code quality and user experience. Each level of the quality scale consists of a list of requirements. t If an integration matches all requirements, it's considered to have reached that level. |
| This is a solid integration that is able to survive poor conditions and can be configured via the user interface. | ||
|
|
||
| - Configurable via config entries. | ||
| - Don't allow configuring already configired device/service (example: no 2 entries for same hub) |
There was a problem hiding this comment.
configired -> configured
|
|
||
| # Platinum 🏆 | ||
|
|
||
| Best of the best. The integration is completely async, meaning it's super fast. Integrations that reach platinum level will require approval by the code owner for each PR. |
There was a problem hiding this comment.
Potentially we'll get stale PRs that are waiting for approval by code owner who's gone in hiding.
There was a problem hiding this comment.
In that case we can downgrade a component to Gold and unblock the PRs.
MartinHjelmare
approved these changes
Sep 26, 2018
|
Going to merge this now. This is not the end of the discussion, but the beginning. Now we have something we can iterate upon. |
andyb2000
added a commit
to andyb2000/developers.home-assistant
that referenced
this pull request
Nov 4, 2024
…nal git library As in issue home-assistant#92 (home-assistant/actions#92) this update to documentation corrects this in line with hassfest validation.
5 tasks
MartinHjelmare
pushed a commit
that referenced
this pull request
Nov 4, 2024
As in issue #92 (home-assistant/actions#92) this update to documentation corrects this in line with hassfest validation.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Preview page: https://deploy-preview-92--developers-home-assistant.netlify.com/docs/en/next/integration_quality_scale_index.html
Fixes home-assistant/architecture#20