Update issue labelling guidelines in README

Add "prevalence" labels to use as a guideline for how often issues crop up.

Combine prevalence and severity to indicate correspondence to how defects
should be prioritised by the team.
This commit is contained in:
Ekaterina Gerasimova 2021-07-29 16:17:04 +01:00
parent 2504c062f1
commit c90430954b

View file

@ -415,19 +415,24 @@ We strive to completely cover all applicable issues with these core labels:
* __[S-Tolerable](https://github.com/vector-im/element-web/labels/S-Tolerable):__ * __[S-Tolerable](https://github.com/vector-im/element-web/labels/S-Tolerable):__
Purely cosmetic or low / no impact to users Purely cosmetic or low / no impact to users
3. __Priority__ — All issues which are not `T-Other` are assigned a priority: 3. __Prevalence__ — All issues labeled `T-Defect` are also assigned a prevalence:
* __[P1](https://github.com/vector-im/element-web/labels/P1):__ Next * __[P-High](https://github.com/vector-im/element-web/labels/P-High):__ Affects most users regularly or impacts most users' first experience (e.g. registration)
* __[P2](https://github.com/vector-im/element-web/labels/P2):__ Later * __[P-Medium](https://github.com/vector-im/element-web/labels/P-Medium):__ Affects some users regularly or most users rarely
* __[P3](https://github.com/vector-im/element-web/labels/P3):__ Eventually * __[P-Low](https://github.com/vector-im/element-web/labels/P-Low):__ Most users are unlikely to come across this or it is a corner case
* __[P4](https://github.com/vector-im/element-web/labels/P4):__ Interesting —
Not yet scheduled, will accept patches
* __[P5](https://github.com/vector-im/element-web/labels/P5):__ Dubious —
Will not schedule, would consider patches
4. __Area__ — Most issues are assigned one or several "areas" using one of the 4. __Area__ — Most issues are assigned one or several "areas" using one of the
many `A-` prefixed labels, e.g. `A-Composer` or `A-Spaces`. Each area label many `A-` prefixed labels, e.g. `A-Composer` or `A-Spaces`. Each area label
maps to a group of features or portion of the UI surface in the app. maps to a group of features or portion of the UI surface in the app.
The Priority label is now deprecated. Defects will now be classed based on their severity and frequency
| Labels | Equivalent priority | What it means |
| ----------- | ----------- | ----------- |
| SCritical&nbsp;and&nbsp;FHigh<br />SCritical&nbsp;and&nbsp;FMedium<br />SMajor&nbsp;and&nbsp;FHigh | P1 | These issues should be worked on in this sprint or next sprint. If the backlog of issues is too long, we should reevaluate why the bugs are not caught earlier. |
| SCritical and FLow<br />SMajor and FMedium<br />SMinor and FHigh | P2 | When all the highest priority bugs are done, this is the next set to tackle. Ideally we should be fixing a few issues from this group every week. |
| SMajor and FLow<br />SMinor and FMedium<br />STolerable and FHigh | P3 | These issues are wishful thinking. We hope to get to them one day, but we they are low priority. There are likely to be some good new contributor issues in here. |
| SMinor and FLow<br />STolerable&nbsp;and&nbsp;FMedium<br />SMinor and FLow | P4 and P5 | These issues are unlikely to be actively looked at by the webapp team, but may be picked up by community. |
### Other common labels ### Other common labels
We have a handful of other labels which are added on an as-needed basis, and not expected to be exhaustive: We have a handful of other labels which are added on an as-needed basis, and not expected to be exhaustive: