Bug 267176 - articles/problem-reports: Remove 'add patch keyword' guideline
Summary: articles/problem-reports: Remove 'add patch keyword' guideline
Status: Closed FIXED
Alias: None
Product: Documentation
Classification: Unclassified
Component: Books & Articles (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Graham Perrin
URL: https://github.com/freebsd/freebsd-do...
Keywords: easy
Depends on:
Blocks:
 
Reported: 2022-10-18 18:01 UTC by Graham Perrin
Modified: 2022-10-29 03:48 UTC (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Graham Perrin freebsd_committer freebsd_triage 2022-10-18 18:01:47 UTC
The article pleads for use of a keyword: 

    patch

<https://docs.freebsd.org/en/articles/problem-reports/#pr-writing-tips>: 

> … If you have a patch, say so. A PR with a patch included is much 
> more likely to be looked at than one without. Please set the 
> patch Keyword in Bugzilla. …


Deprecation was not discovered until after it was batch-added to a number of open bug reports, which had '[patch]' in the summary line but lacked the patch (or patch-ready) keyword. 

Corrections, to signify deprecation, have since been made to: 

    <https://bugs.freebsd.org/bugzilla/describekeywords.cgi>
    <https://wiki.freebsd.org/Bugzilla/Keywords>

Let's correct and update this article, beginning with the misdirection to use the patch keyword. 

See also: 

<https://wiki.freebsd.org/Bugzilla/DosAndDonts>

> …
> 
> Please, do not:
> 
> * use [TAGS] like [MAINTAINER], [UPDATE], [EXP-RUN] and [PATCH] 
>   in issue titles
> 
>   – [NEW PORT] in title for new port submissions is OK for now. 
> 
> …

<https://wiki.freebsd.org/Bugzilla/ThePerfectPortsIssue> 

<https://wiki.freebsd.org/KubilayKocak/Bugzilla/ProblemsAndIdeas> 

<https://wiki.freebsd.org/KubilayKocak/Bugzilla/Roadmap> 

…
Comment 1 Kubilay Kocak freebsd_committer freebsd_triage 2022-10-18 22:32:47 UTC
@Graham +1 on removing a simple and now incorrect statement to ask for keywords, but let's focus on replacing the existing bug reporting guidelines wholesale, as it remains mostly vestigial from the GNATS days.

Can you add a patch here to remove the specific statement to add the patch keyword
Comment 2 Graham Perrin freebsd_committer freebsd_triage 2022-10-19 00:13:58 UTC
(In reply to Kubilay Kocak from comment #1)

Thanks, what's GNATS?
Comment 3 Mark Linimon freebsd_committer freebsd_triage 2022-10-21 02:11:13 UTC
(In reply to Graham Perrin from comment #2)

What FreeBSD used before switching to Bugzilla.  "You're right ... better not to know."
Comment 4 Graham Perrin freebsd_committer freebsd_triage 2022-10-22 16:28:24 UTC
(In reply to Kubilay Kocak from comment #1)

Thanks. Diff in Phabricator, subject to review. 

I'll delete my bug-267176 branch <https://github.com/grahamperrin/freebsd-doc/tree/bug-267176> when this bug is closed.
Comment 5 commit-hook freebsd_committer freebsd_triage 2022-10-29 03:31:29 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/doc/commit/?id=03d204adfdf1a7bb586db28e9e83e63ca683a3ad

commit 03d204adfdf1a7bb586db28e9e83e63ca683a3ad
Author:     Graham Perrin <grahamperrin@FreeBSD.org>
AuthorDate: 2022-10-29 03:16:34 +0000
Commit:     Graham Perrin <grahamperrin@FreeBSD.org>
CommitDate: 2022-10-29 03:16:34 +0000

    articles/problem-reports: cease recommending the 'patch' keyword

    'Writing FreeBSD Problem Reports' pleads for use of a keyword:

        patch

    – https://docs.freebsd.org/en/articles/problem-reports/#pr-writing-tips

    Keywords 'patch' and 'patch-ready' remain in Bugzilla –
    https://bugs.freebsd.org/bugzilla/describekeywords.cgi
    – but should no longer be used. Both keywords are deprecated:

    https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=227147

    Update the article:

    * end the plea
    * explain why the available keywords should not be used.

    PR:                                  267176
    Bug report: https://bugs.freebsd.org/267176
    Mentors:               gjb, carlavilla
    Reviewed by:           gjb
    Approved by:           gjb
    Differential revision: https://reviews.freebsd.org/D37086

 documentation/content/en/articles/problem-reports/_index.adoc | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)