{"title":"Triaging tickets","version":"6.0","locale":"it","docname":"internals/contributing/triaging-tickets","url":"/it/6.0/internals/contributing/triaging-tickets/","canonical":"https://djangodocs.dev/it/6.0/internals/contributing/triaging-tickets/","summary":"Django uses Trac for managing the work on the code base. Trac is a community-tended garden of the bugs people have found and the features Django has decided to add.…","html":"<h1>Triaging tickets<a class=\"heading-anchor\" href=\"#triaging-tickets\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h1>\n<p>Django uses <a class=\"reference external\" href=\"https://code.djangoproject.com/\">Trac</a> for managing the work on the code base. Trac is a\ncommunity-tended garden of the bugs people have found and the features Django\nhas decided to add. As in any garden, sometimes there are weeds to be pulled\nand sometimes there are flowers and vegetables that need picking. We need your\nhelp to sort out one from the other, and in the end, we all benefit together.</p>\n<p>Like all gardens, we can aspire to perfection, but in reality there’s no such\nthing. Even in the most pristine garden there are still snails and insects.\nIn a community garden there are also helpful people who – with the best of\nintentions – fertilize the weeds and poison the roses. It’s the job of the\ncommunity as a whole to self-manage, keep the problems to a minimum, and\neducate those coming into the community so that they can become valuable\ncontributing members.</p>\n<p>Similarly, while we aim for Trac to be a perfect representation of the state of\nDjango’s progress, we acknowledge that this will not happen. By distributing\nthe load of Trac maintenance to the community, we accept that there will be\nmistakes. Trac is «mostly accurate», and we give allowances for the fact that\nsometimes it will be wrong. That’s okay. We’re perfectionists with deadlines.</p>\n<p>We rely on the community to keep participating, keep tickets as accurate as\npossible, and raise issues for discussion on the <a class=\"reference external\" href=\"https://forum.djangoproject.com/\">Django Forum</a> when there is\nconfusion or disagreement.</p>\n<p>Django is a community project, and every contribution helps. We can’t do this\nwithout <strong>you</strong>!</p>\n<section id=\"triage-workflow\">\n<span id=\"id1\"></span><h2>Triage workflow<a class=\"heading-anchor\" href=\"#triage-workflow\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h2>\n<p>Unfortunately, not all reports in the ticket tracker provide all the\n<a class=\"reference internal\" href=\"/it/6.0/internals/contributing/bugs-and-features/#reporting-bugs\"><span class=\"std std-ref\">required details</span></a>. A number of tickets have proposed\nsolutions, but those don’t necessarily meet all the requirements <a class=\"reference internal\" href=\"/it/6.0/internals/contributing/writing-code/submitting-patches/#patch-style\"><span class=\"std std-ref\">adhering\nto the guidelines for contributing</span></a>.</p>\n<p>One way to help out is to <em>triage</em> tickets that have been created by other\nusers.</p>\n<p>Most of the workflow is based around the concept of a ticket’s\n<a class=\"reference internal\" href=\"#triage-stages\"><span class=\"std std-ref\">triage stages</span></a>. Each stage describes where in its\nlifetime a given ticket is at any time. Along with a handful of flags, this\nattribute easily tells us what and who each ticket is waiting on.</p>\n<p>Since a picture is worth a thousand words, let’s start there:</p>\n<img alt=\"Django's ticket triage workflow\" src=\"/it/6.0/_images/contribution_process.svg\" />\n<p>We have four roles in this diagram. Maintainers (also known as Fellows) usually\ntake part in all of them, but anyone in the Django community can participate in\nany role except merger. The <a class=\"reference internal\" href=\"/it/6.0/internals/organization/#mergers-team\"><span class=\"std std-ref\">merger role</span></a> is granted by a\nvote of the <a class=\"reference internal\" href=\"/it/6.0/internals/organization/#steering-council\"><span class=\"std std-ref\">Steering Council</span></a>.</p>\n<ul class=\"simple\">\n<li><p>Triagers: anyone can take on this role by checking whether a ticket describes\na real issue and keeping the tracker organized.</p></li>\n<li><p>Bug fixers: anyone can contribute by opening a pull request and working on a\nsolution for a ticket.</p></li>\n<li><p>Reviewers: anyone can review pull requests and suggest improvements.</p></li>\n<li><p>Mergers: people with commit access who make the final decision to merge a\nchange.</p></li>\n</ul>\n<p>Our Trac system is intentionally open to the public, and anyone can help by\nworking on tickets. Django is a community project, and we encourage\n<a class=\"reference internal\" href=\"#how-can-i-help-with-triaging\"><span class=\"std std-ref\">triage and collaboration by the community</span></a>. This could be you!</p>\n<p>For example, here’s the typical lifecycle of a ticket:</p>\n<ul class=\"simple\">\n<li><p>Alice creates a ticket and opens an incomplete pull request (missing tests,\nincorrect implementation).</p></li>\n<li><p>Bob reviews the pull request, marks the ticket as «Accepted», sets the\nflags «needs tests» and «patch needs improvement», and leaves a comment\nexplaining how Alice can improve the patch. This puts the ticket\nautomatically into the «waiting on author» queue within the «accepted» stage.</p></li>\n<li><p>Alice updates the pull request, adding tests (but not yet fixing the\nimplementation), and removes the two flags. The ticket moves into the «needs\nPR review» queue.</p></li>\n<li><p>Charlie reviews the pull request, sets the «patch needs improvement» flag\nagain, and leaves another comment suggesting changes to the implementation.\nThe ticket moves back to the «waiting on author» queue.</p></li>\n<li><p>Alice updates the pull request again, this time fixing the implementation,\nand removes the «patch needs improvement» flag. The ticket moves once more\ninto the «needs PR review» queue.</p></li>\n<li><p>Daisy reviews the pull request and marks the ticket as «Ready for checkin».</p></li>\n<li><p>Jacob, a <a class=\"reference internal\" href=\"/it/6.0/internals/organization/#mergers-team\"><span class=\"std std-ref\">merger</span></a>, reviews and merges the pull request.</p></li>\n</ul>\n<p>Some tickets move through these steps quickly, while others take more time and\ndiscussion. Each contribution helps Django improve.</p>\n</section>\n<section id=\"triage-stages\">\n<span id=\"id2\"></span><h2>Triage stages<a class=\"heading-anchor\" href=\"#triage-stages\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h2>\n<p>Below we describe in more detail the various stages that a ticket may flow\nthrough during its lifetime.</p>\n<section id=\"unreviewed\">\n<h3>Unreviewed<a class=\"heading-anchor\" href=\"#unreviewed\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>The ticket has not been reviewed by anyone who felt qualified to make a\njudgment about whether the ticket contained a valid issue or ought to be closed\nfor any reasons. Unreviewed tickets appear in the «triage» queue.</p>\n<p>Unreviewed tickets may receive additional refinement before being accepted.\nUnless you are both the author of the ticket and intending to submit a patch,\nunreviewed tickets should not be <a class=\"reference internal\" href=\"/it/6.0/internals/contributing/writing-code/submitting-patches/#claiming-tickets\"><span class=\"std std-ref\">claimed</span></a>.</p>\n</section>\n<section id=\"accepted\">\n<h3>Accepted<a class=\"heading-anchor\" href=\"#accepted\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>The absolute meaning of «accepted» is that the issue described in the ticket is\nvalid and actionable. It is broken out into three queues:</p>\n<ul>\n<li><p><strong>Needs Patch</strong> (Accepted + No Flags)</p>\n<p>The ticket is valid, but no one has submitted a patch for it yet. Often this\nmeans you could safely start writing a fix for it. This is generally more\ntrue for the case of accepted bugs than accepted features. A ticket for a bug\nthat has been accepted means that the issue has been verified by at least one\ntriager as a legitimate bug - and should probably be fixed if possible.</p>\n<p>For new features, accepted tickets should only exist after the idea has gone\nthrough the appropriate <a class=\"reference internal\" href=\"/it/6.0/internals/contributing/bugs-and-features/#requesting-features\"><span class=\"std std-ref\">process for suggesting new features</span></a> and received community and <a class=\"reference internal\" href=\"/it/6.0/internals/organization/#steering-council\"><span class=\"std std-ref\">Steering Council</span></a> approval, or been accepted in a DEP.</p>\n</li>\n<li><p><strong>Needs PR Review</strong> (Accepted + Has Patch)</p>\n<p>The ticket is waiting for people to review the supplied solution. This means\ndownloading the patch and trying it out, verifying that it contains tests\nand docs, running the test suite with the included patch, and leaving\nfeedback on the ticket.</p>\n</li>\n<li><p><strong>Waiting On Author</strong> (Accepted + Has Patch + Needs fixes)</p>\n<p>This means the ticket has been reviewed, and has been found to need further\nwork. «Needs tests» and «Needs documentation» are self-explanatory. «Patch\nneeds improvement» will generally be accompanied by a comment on the ticket\nexplaining what is needed to improve the code.</p>\n</li>\n</ul>\n</section>\n<section id=\"ready-for-checkin\">\n<h3>Ready For Checkin<a class=\"heading-anchor\" href=\"#ready-for-checkin\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>The ticket was reviewed by any member of the community other than the person\nwho supplied the patch and found to meet all the requirements for a\ncommit-ready contribution. A <a class=\"reference internal\" href=\"/it/6.0/internals/organization/#mergers-team\"><span class=\"std std-ref\">merger</span></a> now needs to give\na final review prior to being committed.</p>\n<p>There are a lot of pull requests. It can take a while for your patch to get\nreviewed. See the <a class=\"reference internal\" href=\"/it/6.0/faq/contributing/#new-contributors-faq\"><span class=\"std std-ref\">contributing code FAQ</span></a> for some\nideas here.</p>\n</section>\n<section id=\"someday-maybe\">\n<h3>Someday/Maybe<a class=\"heading-anchor\" href=\"#someday-maybe\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>This stage isn’t shown on the diagram. It’s used sparingly to keep track of\nlong-term changes.</p>\n<p>These tickets are uncommon and overall less useful since they don’t describe\nconcrete actionable issues.</p>\n</section>\n</section>\n<section id=\"other-triage-attributes\">\n<h2>Other triage attributes<a class=\"heading-anchor\" href=\"#other-triage-attributes\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h2>\n<p>A number of flags, appearing as checkboxes in Trac, can be set on a ticket:</p>\n<section id=\"has-patch\">\n<h3>Has patch<a class=\"heading-anchor\" href=\"#has-patch\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>This means the ticket has an associated solution. These will be reviewed to\nensure they adhere to the <a class=\"reference internal\" href=\"/it/6.0/internals/contributing/writing-code/submitting-patches/#patch-review-checklist\"><span class=\"std std-ref\">documented guidelines</span></a>.</p>\n<p>The following three fields (Needs documentation, Needs tests,\nPatch needs improvement) apply only if a patch has been supplied.</p>\n</section>\n<section id=\"needs-documentation\">\n<h3>Needs documentation<a class=\"heading-anchor\" href=\"#needs-documentation\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>This flag is used for tickets with patches that need associated\ndocumentation. Complete documentation of features is a prerequisite\nbefore we can check them into the codebase.</p>\n</section>\n<section id=\"needs-tests\">\n<h3>Needs tests<a class=\"heading-anchor\" href=\"#needs-tests\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>This flags the patch as needing associated unit tests. Again, this\nis a required part of a valid contribution.</p>\n</section>\n<section id=\"patch-needs-improvement\">\n<h3>Patch needs improvement<a class=\"heading-anchor\" href=\"#patch-needs-improvement\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>This flag means that although the ticket <em>has</em> a solution, it’s not quite\nready for checkin. This could mean the patch no longer applies\ncleanly, there is a flaw in the implementation, or that the code\ndoesn’t meet our standards.</p>\n</section>\n<section id=\"easy-pickings\">\n<h3>Easy pickings<a class=\"heading-anchor\" href=\"#easy-pickings\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>Tickets that would require small, easy, changes.</p>\n</section>\n<section id=\"type\">\n<h3>Type<a class=\"heading-anchor\" href=\"#type\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>Tickets should be categorized by <em>type</em> between:</p>\n<ul class=\"simple\">\n<li><dl class=\"simple\">\n<dt>New Feature</dt><dd><p>For adding something new.</p>\n</dd>\n</dl>\n</li>\n<li><dl class=\"simple\">\n<dt>Bug</dt><dd><p>For when an existing thing is broken or not behaving as expected.</p>\n</dd>\n</dl>\n</li>\n<li><dl class=\"simple\">\n<dt>Cleanup/optimization</dt><dd><p>For when nothing is broken but something could be made cleaner,\nbetter, faster, stronger.</p>\n</dd>\n</dl>\n</li>\n</ul>\n</section>\n<section id=\"component\">\n<h3>Component<a class=\"heading-anchor\" href=\"#component\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>Tickets should be classified into <em>components</em> indicating which area of\nthe Django codebase they belong to. This makes tickets better organized and\neasier to find.</p>\n</section>\n<section id=\"severity\">\n<h3>Severity<a class=\"heading-anchor\" href=\"#severity\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>The <em>severity</em> attribute is used to identify blockers, that is, issues that\nshould get fixed before releasing the next version of Django. Typically those\nissues are bugs causing regressions from earlier versions or potentially\ncausing severe data losses. This attribute is quite rarely used and the vast\nmajority of tickets have a severity of «Normal».</p>\n</section>\n<section id=\"version\">\n<h3>Version<a class=\"heading-anchor\" href=\"#version\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>The <em>version</em> attribute indicates the earliest version in which the bug was\nreproduced. During triage, this field can be updated, but there is no need to\nmake further updates when that version goes out of support. The field should\nnot be reset to «dev» to show the issue still exists: instead, the tested\ncommit hash can be noted in a comment.</p>\n</section>\n<section id=\"ui-ux\">\n<h3>UI/UX<a class=\"heading-anchor\" href=\"#ui-ux\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>This flag is used for tickets that relate to User Interface and User\nExperiences questions. For example, this flag would be appropriate for\nuser-facing features in forms or the admin interface.</p>\n</section>\n<section id=\"cc\">\n<h3>Cc<a class=\"heading-anchor\" href=\"#cc\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>You may add your username or email address to this field to be notified when\nnew contributions are made to the ticket.</p>\n</section>\n<section id=\"keywords\">\n<h3>Keywords<a class=\"heading-anchor\" href=\"#keywords\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h3>\n<p>With this field you may label a ticket with multiple keywords. This can be\nuseful, for example, to group several tickets on the same theme. Keywords can\neither be comma or space separated. Keyword search finds the keyword string\nanywhere in the keywords. For example, clicking on a ticket with the keyword\n«form» will yield similar tickets tagged with keywords containing strings such\nas «formset», «modelformset», and «ManagementForm».</p>\n</section>\n</section>\n<section id=\"closing-tickets\">\n<span id=\"id3\"></span><h2>Closing Tickets<a class=\"heading-anchor\" href=\"#closing-tickets\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h2>\n<p>When a ticket has completed its useful lifecycle, it’s time for it to be\nclosed. Closing a ticket is a big responsibility, though. You have to be sure\nthat the issue is really resolved, and you need to keep in mind that the\nreporter of the ticket may not be happy to have their ticket closed (unless\nit’s fixed!). If you’re not certain about closing a ticket, leave a comment\nwith your thoughts instead.</p>\n<p>If you do close a ticket, you should always make sure of the following:</p>\n<ul class=\"simple\">\n<li><p>Be certain that the issue is resolved.</p></li>\n<li><p>Leave a comment explaining the decision to close the ticket.</p></li>\n<li><p>If there is a way they can improve the ticket to reopen it, let them know.</p></li>\n<li><p>If the ticket is a duplicate, reference the original ticket. Also\ncross-reference the closed ticket by leaving a comment in the original one\n– this allows to access more related information about the reported bug\nor requested feature.</p></li>\n<li><p><strong>Be polite.</strong> No one likes having their ticket closed. It can be\nfrustrating or even discouraging. The best way to avoid turning people\noff from contributing to Django is to be polite and friendly and to offer\nsuggestions for how they could improve this ticket and other tickets in\nthe future.</p></li>\n</ul>\n<p>A ticket can be resolved in a number of ways:</p>\n<ul class=\"simple\">\n<li><dl class=\"simple\">\n<dt>fixed</dt><dd><p>Used once a patch has been rolled into Django and the issue is fixed.</p>\n</dd>\n</dl>\n</li>\n<li><dl class=\"simple\">\n<dt>invalid</dt><dd><p>Used if the ticket is found to be incorrect. This means that the\nissue in the ticket is actually the result of a user error, or\ndescribes a problem with something other than Django, or isn’t\na bug report or feature request at all (for example, some new users\nsubmit support queries as tickets).</p>\n</dd>\n</dl>\n</li>\n<li><dl class=\"simple\">\n<dt>wontfix</dt><dd><p>Used when someone decides that the request isn’t appropriate for\nconsideration in Django. Sometimes a ticket is closed as «wontfix» with a\nrequest for the reporter to start a discussion on the <a class=\"reference external\" href=\"https://forum.djangoproject.com/\">Django Forum</a> if\nthey feel differently from the rationale provided by the person who\nclosed the ticket. Other times, a discussion precedes the decision to\nclose a ticket. Always use the forum to get a consensus before reopening\ntickets closed as «wontfix».</p>\n</dd>\n</dl>\n</li>\n<li><dl class=\"simple\">\n<dt>needsnewfeatureprocess</dt><dd><p>Used when the ticket merits a new feature, which will need to get\ncommunity input and support.\nSee the <a class=\"reference internal\" href=\"/it/6.0/internals/contributing/bugs-and-features/#requesting-features\"><span class=\"std std-ref\">process for suggesting new features</span></a>.</p>\n</dd>\n</dl>\n</li>\n<li><dl class=\"simple\">\n<dt>duplicate</dt><dd><p>Used when another ticket covers the same issue. By closing duplicate\ntickets, we keep all the discussion in one place, which helps\neveryone.</p>\n</dd>\n</dl>\n</li>\n<li><dl class=\"simple\">\n<dt>worksforme</dt><dd><p>Used when the ticket doesn’t contain enough detail to replicate\nthe original bug.</p>\n</dd>\n</dl>\n</li>\n<li><dl class=\"simple\">\n<dt>needsinfo</dt><dd><p>Used when the ticket does not contain enough information to replicate\nthe reported issue but is potentially still valid. The ticket\nshould be reopened when more information is supplied.</p>\n</dd>\n</dl>\n</li>\n</ul>\n<p>If you believe that the ticket was closed in error – because you’re\nstill having the issue, or it’s popped up somewhere else, or the triagers have\nmade a mistake – please reopen the ticket and provide further information.\nAgain, please do not reopen tickets that have been marked as «wontfix» or\n«needsnewfeatureprocess». For «wontfix» tickets, bring the issue to the\n<a class=\"reference external\" href=\"https://forum.djangoproject.com/\">Django Forum</a> instead. For «needsnewfeatureprocess» tickets, propose the\nfeature through the <a class=\"reference internal\" href=\"/it/6.0/internals/contributing/bugs-and-features/#requesting-features\"><span class=\"std std-ref\">new features process</span></a>.</p>\n</section>\n<section id=\"how-can-i-help-with-development\">\n<span id=\"how-can-i-help-with-triaging\"></span><h2>How can I help with development?<a class=\"heading-anchor\" href=\"#how-can-i-help-with-development\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h2>\n<p>The development process is primarily driven by community members. Really,\n<strong>ANYONE</strong> can help.</p>\n<p>To get involved, start by <a class=\"reference external\" href=\"https://www.djangoproject.com/accounts/register/\">creating an account on Trac</a>. If you have an\naccount but have forgotten your password, you can reset it using the <a class=\"reference external\" href=\"https://www.djangoproject.com/accounts/password/reset/\">password\nreset page</a>.</p>\n<p>Then, you can help out by:</p>\n<ul class=\"simple\">\n<li><p>Closing «Unreviewed» tickets as «invalid», «worksforme», «duplicate»,\n«wontfix», or «needsnewfeatureprocess».</p></li>\n<li><p>Closing «Unreviewed» tickets as «needsinfo» when the description is too\nsparse to be actionable.</p></li>\n<li><p>Correcting the «Needs tests», «Needs documentation», or «Has patch»\nflags for tickets where they are incorrectly set.</p></li>\n<li><p>Setting the «<a class=\"reference external\" href=\"https://code.djangoproject.com/query?status=!closed&amp;easy=1\">Easy pickings</a>» flag for tickets that are small and\nrelatively straightforward.</p></li>\n<li><p>Set the <em>type</em> of tickets that are still uncategorized.</p></li>\n<li><p>Checking that old tickets are still valid. If a ticket hasn’t seen\nany activity in a long time, it’s possible that the problem has been\nfixed but the ticket hasn’t yet been closed.</p></li>\n<li><p>Identifying trends and themes in the tickets. If there are a lot of bug\nreports about a particular part of Django, it may indicate we should\nconsider refactoring that part of the code. If a trend is emerging,\nyou should raise it for discussion (referencing the relevant tickets)\non the <a class=\"reference external\" href=\"https://forum.djangoproject.com/\">Django Forum</a>.</p></li>\n<li><p>Verify if solutions submitted by others are correct. If they are correct\nand also contain appropriate documentation and tests then move them to the\n«Ready for Checkin» stage. If they are not correct then leave a comment to\nexplain why and set the corresponding flags («Patch needs improvement»,\n«Needs tests» etc.).</p></li>\n</ul>\n<aside class=\"admonition admonition-note\" role=\"note\">\n<p class=\"admonition-title\">Nota</p>\n<p>The <a class=\"reference external\" href=\"https://code.djangoproject.com/wiki/Reports\">Reports page</a> contains links to many useful Trac queries, including\nseveral that are useful for triaging tickets and reviewing proposals as\nsuggested above.</p>\n<p>You can also find more <a class=\"reference internal\" href=\"/it/6.0/internals/contributing/new-contributors/\"><span class=\"doc\">Advice for new contributors</span></a>.</p>\n</aside>\n<p>However, we do ask the following of all general community members working in\nthe ticket database:</p>\n<ul class=\"simple\">\n<li><p>Please <strong>don’t</strong> promote your own tickets to «Accepted». Another community\nmember should review the report and set this stage after reproducing and\nconfirming the issue.</p></li>\n<li><p>Please <strong>don’t</strong> promote your own tickets to «Ready for checkin». You\nmay mark other people’s tickets that you’ve reviewed as «Ready for\ncheckin», but you should get at minimum one other community member to\nreview a patch that you submit.</p></li>\n<li><p>Please <strong>don’t</strong> reverse a decision without posting a message to the\n<a class=\"reference external\" href=\"https://forum.djangoproject.com/\">Django Forum</a> to find consensus.</p></li>\n<li><p>If you’re unsure if you should be making a change, don’t make the\nchange but instead leave a comment with your concerns on the ticket,\nor post a message to the <a class=\"reference external\" href=\"https://forum.djangoproject.com/\">Django Forum</a>. It’s okay to be unsure, but your\ninput is still valuable.</p></li>\n</ul>\n</section>\n<section id=\"bisecting-a-regression\">\n<span id=\"id5\"></span><h2>Bisecting a regression<a class=\"heading-anchor\" href=\"#bisecting-a-regression\"><span class=\"visually-hidden\">Link to this heading</span><span aria-hidden=\"true\">#</span></a></h2>\n<p>A regression is a bug that’s present in some newer version of Django but not in\nan older one. An extremely helpful piece of information is the commit that\nintroduced the regression. Knowing the commit that caused the change in\nbehavior helps identify if the change was intentional or if it was an\ninadvertent side-effect. Here’s how you can determine this.</p>\n<p>Begin by writing a regression test for Django’s test suite for the issue. For\nexample, we’ll pretend we’re debugging a regression in migrations. After you’ve\nwritten the test and confirmed that it fails on the latest main branch, put it\nin a separate file that you can run standalone. For our example, we’ll pretend\nwe created <code class=\"docutils literal notranslate\"><span class=\"pre\">tests/migrations/test_regression.py</span></code>, which can be run with:</p>\n<div class=\"code-block\" data-language=\"shell\"><div class=\"code-block-toolbar\"><span class=\"code-block-language\">Shell</span><button type=\"button\" class=\"copy-button\" data-copy hidden><span class=\"copy-button-label\">Copy</span></button></div><pre role=\"group\" tabindex=\"0\" aria-label=\"Shell code\"><code>$<span class=\"w\"> </span>./runtests.py<span class=\"w\"> </span>migrations.test_regression\n</code></pre></div>\n<p>Next, we mark the current point in history as being «bad» since the test fails:</p>\n<div class=\"code-block\" data-language=\"shell\"><div class=\"code-block-toolbar\"><span class=\"code-block-language\">Shell</span><button type=\"button\" class=\"copy-button\" data-copy hidden><span class=\"copy-button-label\">Copy</span></button></div><pre role=\"group\" tabindex=\"0\" aria-label=\"Shell code\"><code>$<span class=\"w\"> </span>git<span class=\"w\"> </span>bisect<span class=\"w\"> </span>bad\nYou<span class=\"w\"> </span>need<span class=\"w\"> </span>to<span class=\"w\"> </span>start<span class=\"w\"> </span>by<span class=\"w\"> </span><span class=\"s2\">&quot;git bisect start&quot;</span>\nDo<span class=\"w\"> </span>you<span class=\"w\"> </span>want<span class=\"w\"> </span>me<span class=\"w\"> </span>to<span class=\"w\"> </span><span class=\"k\">do</span><span class=\"w\"> </span>it<span class=\"w\"> </span><span class=\"k\">for</span><span class=\"w\"> </span>you<span class=\"w\"> </span><span class=\"o\">[</span>Y/n<span class=\"o\">]</span>?<span class=\"w\"> </span>y\n</code></pre></div>\n<p>Now, we need to find a point in git history before the regression was\nintroduced (i.e. a point where the test passes). Use something like\n<code class=\"docutils literal notranslate\"><span class=\"pre\">git</span> <span class=\"pre\">checkout</span> <span class=\"pre\">HEAD~100</span></code> to check out an earlier revision (100 commits\nearlier, in this case). Check if the test fails. If so, mark that point as\n«bad» (<code class=\"docutils literal notranslate\"><span class=\"pre\">git</span> <span class=\"pre\">bisect</span> <span class=\"pre\">bad</span></code>), then check out an earlier revision and recheck.\nOnce you find a revision where your test passes, mark it as «good»:</p>\n<div class=\"code-block\" data-language=\"shell\"><div class=\"code-block-toolbar\"><span class=\"code-block-language\">Shell</span><button type=\"button\" class=\"copy-button\" data-copy hidden><span class=\"copy-button-label\">Copy</span></button></div><pre role=\"group\" tabindex=\"0\" aria-label=\"Shell code\"><code>$<span class=\"w\"> </span>git<span class=\"w\"> </span>bisect<span class=\"w\"> </span>good\nBisecting:<span class=\"w\"> </span>X<span class=\"w\"> </span>revisions<span class=\"w\"> </span>left<span class=\"w\"> </span>to<span class=\"w\"> </span><span class=\"nb\">test</span><span class=\"w\"> </span>after<span class=\"w\"> </span>this<span class=\"w\"> </span><span class=\"o\">(</span>roughly<span class=\"w\"> </span>Y<span class=\"w\"> </span>steps<span class=\"o\">)</span>\n...\n</code></pre></div>\n<p>Now we’re ready for the fun part: using <code class=\"docutils literal notranslate\"><span class=\"pre\">git</span> <span class=\"pre\">bisect</span> <span class=\"pre\">run</span></code> to automate the rest\nof the process:</p>\n<div class=\"code-block\" data-language=\"shell\"><div class=\"code-block-toolbar\"><span class=\"code-block-language\">Shell</span><button type=\"button\" class=\"copy-button\" data-copy hidden><span class=\"copy-button-label\">Copy</span></button></div><pre role=\"group\" tabindex=\"0\" aria-label=\"Shell code\"><code>$<span class=\"w\"> </span>git<span class=\"w\"> </span>bisect<span class=\"w\"> </span>run<span class=\"w\"> </span>tests/runtests.py<span class=\"w\"> </span>migrations.test_regression\n</code></pre></div>\n<p>You should see <code class=\"docutils literal notranslate\"><span class=\"pre\">git</span> <span class=\"pre\">bisect</span></code> use a binary search to automatically checkout\nrevisions between the good and bad commits until it finds the first «bad»\ncommit where the test fails.</p>\n<p>Now, report your results on the Trac ticket, and please include the regression\ntest as an attachment. When someone writes a fix for the bug, they’ll already\nhave your test as a starting point.</p>\n</section>","rootId":"triaging-tickets","toc":[{"title":"Triage workflow","anchor":"triage-workflow","children":[]},{"title":"Triage stages","anchor":"triage-stages","children":[{"title":"Unreviewed","anchor":"unreviewed","children":[]},{"title":"Accepted","anchor":"accepted","children":[]},{"title":"Ready For Checkin","anchor":"ready-for-checkin","children":[]},{"title":"Someday/Maybe","anchor":"someday-maybe","children":[]}]},{"title":"Other triage attributes","anchor":"other-triage-attributes","children":[{"title":"Has patch","anchor":"has-patch","children":[]},{"title":"Needs documentation","anchor":"needs-documentation","children":[]},{"title":"Needs tests","anchor":"needs-tests","children":[]},{"title":"Patch needs improvement","anchor":"patch-needs-improvement","children":[]},{"title":"Easy pickings","anchor":"easy-pickings","children":[]},{"title":"Type","anchor":"type","children":[]},{"title":"Component","anchor":"component","children":[]},{"title":"Severity","anchor":"severity","children":[]},{"title":"Version","anchor":"version","children":[]},{"title":"UI/UX","anchor":"ui-ux","children":[]},{"title":"Cc","anchor":"cc","children":[]},{"title":"Keywords","anchor":"keywords","children":[]}]},{"title":"Closing Tickets","anchor":"closing-tickets","children":[]},{"title":"How can I help with development?","anchor":"how-can-i-help-with-development","children":[]},{"title":"Bisecting a regression","anchor":"bisecting-a-regression","children":[]}],"breadcrumbs":[{"docname":"internals/index","title":"Django internals","url":"/it/6.0/internals/"},{"docname":"internals/contributing/index","title":"Contributing to Django","url":"/it/6.0/internals/contributing/"}],"prev":{"docname":"internals/contributing/bugs-and-features","title":"Reporting bugs and requesting features","url":"/it/6.0/internals/contributing/bugs-and-features/"},"next":{"docname":"internals/mailing-lists","title":"Mailing lists and Forum","url":"/it/6.0/internals/mailing-lists/"},"formats":{"html":"/it/6.0/internals/contributing/triaging-tickets/","markdown":"/it/6.0/internals/contributing/triaging-tickets.md","json":"/it/6.0/internals/contributing/triaging-tickets.json"},"source":"https://github.com/django/django/blob/stable/6.0.x/docs/internals/contributing/triaging-tickets.txt","official":"https://docs.djangoproject.com/it/6.0/internals/contributing/triaging-tickets/","inVersions":["6.1","6.0","5.2","5.1","5.0","4.2","4.1","4.0","3.2"],"inLocales":["en","sv","zh-hans","ga","fr","ja","id","it","pt-br","ko","es","el","pl"]}