Differences
This shows you the differences between two versions of the page.
| Both sides previous revision Previous revision Next revision | Previous revision | ||
| bugzilla:dosanddonts [2023/10/12 06:32] – Hmmph. The visual bug affecting anchors is invisible when using the Preview feature of MoinMoin. Let's try, again, to work around … GrahamPerrin | bugzilla:dosanddonts [2023/10/14 08:40] (current) – Consistency: remove superfluous leading white space from list items (bullet points). GrahamPerrin | ||
|---|---|---|---|
| Line 13: | Line 13: | ||
| * [[https:// | * [[https:// | ||
| * before submitting (saving), double-check the ' | * before submitting (saving), double-check the ' | ||
| - | * you cannot preview what will become your opening comment — [[https:// | + | * you cannot preview what will become your opening comment — [[https:// |
| - | * saved commentary can be neither edited, nor deleted — [[https:// | + | * saved commentary can be neither edited, nor deleted — [[https:// |
| * if the description involves an attachment that aims to patch '' | * if the description involves an attachment that aims to patch '' | ||
| Line 63: | Line 63: | ||
| * **Note**: maintainer approval requests must include the maintainer' | * **Note**: maintainer approval requests must include the maintainer' | ||
| * post a comment if you're unable to provide requested feedback (eg. due to lack of time or even loss of interest) | * post a comment if you're unable to provide requested feedback (eg. due to lack of time or even loss of interest) | ||
| - | * be polite — say //please//, //thank you//, and so on | + | * be polite — say //please//, //thank you//, and so on |
| * be patient | * be patient | ||
| * the ' | * the ' | ||
| * // | * // | ||
| * if //visible// work towards a fix takes longer than you would like, remember that most work is volunteered, | * if //visible// work towards a fix takes longer than you would like, remember that most work is volunteered, | ||
| - | * remember the [[https:// | + | * |
| * treat others as you would like others to treat you — what you wish upon others, you wish upon yourself | * treat others as you would like others to treat you — what you wish upon others, you wish upon yourself | ||
| * apply the rule equally in areas that run parallel to Bugzilla, such as [[https:// | * apply the rule equally in areas that run parallel to Bugzilla, such as [[https:// | ||
| Line 95: | Line 95: | ||
| Please, do not: | Please, do not: | ||
| - | * use the //patch// or // | + | * use the //patch// or // |
| - | * use **[TAGS]** like [MAINTAINER], [UPDATE], [EXP-RUN] and [PATCH] in issue titles | + | * use '' |
| - | * [NEW PORT] in title for new port submissions **is OK** for now. | + | * '' |
| - | * use an **empty summary**. | + | * use an **empty summary** |
| - | * use **foo is broken** as a summary. | + | * use **foo is broken** as a summary |
| - | * add just **me too** comments. DO add comments if you have additional information. | + | * add just **me too** comments. DO add comments if you have additional information |
| * **bump** issues (add " | * **bump** issues (add " | ||
| * instead, //do// update issue metadata to bring it up to date | * instead, //do// update issue metadata to bring it up to date | ||
| - | * //do// include comments if they clarify what is now needed to progress the issue. | + | * //do// include comments if they clarify what is now needed to progress the issue |
| - | * **speculate** on what an issue/bug is caused by. Stick to facts and evidence. | + | * **speculate** on what an issue/bug is caused by. Stick to facts and evidence |
| * **fiddle with Attachment** mime types | * **fiddle with Attachment** mime types | ||
| - | * cancel/ | + | * cancel/ |
| - | * **paste large logs** or file contents in comments (DO: use // | + | * **paste large logs** or file contents in comments (DO: use // |
| - | * include poudriere/ | + | * include poudriere/ |
| - | * **wait for others** to add or update issue fields. Many hands make light work. | + | * **wait for others** to add or update issue fields. Many hands make light work |
| * add a //Comment// if the change made is self-explanatory. Example: | * add a //Comment// if the change made is self-explanatory. Example: | ||
| Line 122: | Line 122: | ||
| * try to **game the system**. Example: by adding easy patch-ready or setting approved flags | * try to **game the system**. Example: by adding easy patch-ready or setting approved flags | ||
| * use maintainer-**feedback** flag to declare approval of a patch. (DO use the maintainer-**approval** flag) | * use maintainer-**feedback** flag to declare approval of a patch. (DO use the maintainer-**approval** flag) | ||
| - | * **combine or include patches for multiple ports** in the same issue, unless they *must* be committed together. | + | * **combine or include patches for multiple ports** in the same issue, unless they *must* be committed together |
| * instead, create separate issues and have them //Depend on// or //Block// each other as appropriate | * instead, create separate issues and have them //Depend on// or //Block// each other as appropriate | ||
| * change the status of an issue to past Open (eg: In Progress) unless and until it has a real (not a team or mailing list) Assignee | * change the status of an issue to past Open (eg: In Progress) unless and until it has a real (not a team or mailing list) Assignee | ||
| * this criterion does not suit all workflows, should probably be mentioned at [[KubilayKocak/ | * this criterion does not suit all workflows, should probably be mentioned at [[KubilayKocak/ | ||
| * close (resolve) an issue until all commits/ | * close (resolve) an issue until all commits/ | ||
| - | * use **external URL' | + | * use **external URL' |
| - | * instead, include URL contents as attachments with the URL as the Description, | + | * instead, include URL contents as attachments with the URL as the Description, |
| - | * use the same PR to do multiple unrelated things. E.g: if while working on a **fix** for a port a new version is released, create a new PR to update the port. | + | * use the same PR to do multiple unrelated things. E.g: if while working on a **fix** for a port a new version is released, create a new PR to update the port |
| - | * include **Signatures** at the bottom/end of issue reports or comments. | + | * include **Signatures** at the bottom/end of issue reports or comments |
| - | * be unnecessarily verbose. | + | * be unnecessarily verbose |
| - | * create, or engage in, **argument** | + | * create, or engage in, argument |
| - | * instead, be factual. | + | * instead, be factual |
| + | * disagreement is natural — when a technical difference of opinion arises, do, please, build and maintain mutual respect (remember the [[# | ||
| * be abusive, or use abusive language — commentary is [[# | * be abusive, or use abusive language — commentary is [[# | ||