<?xml version="1.0"?>
<feed xmlns="http://www.w3.org/2005/Atom" xml:lang="en">
	<id>https://docs.moodle.org/310/en/api.php?action=feedcontributions&amp;feedformat=atom&amp;user=Alicethomas</id>
	<title>MoodleDocs - User contributions [en]</title>
	<link rel="self" type="application/atom+xml" href="https://docs.moodle.org/310/en/api.php?action=feedcontributions&amp;feedformat=atom&amp;user=Alicethomas"/>
	<link rel="alternate" type="text/html" href="https://docs.moodle.org/310/en/Special:Contributions/Alicethomas"/>
	<updated>2026-04-21T15:48:19Z</updated>
	<subtitle>User contributions</subtitle>
	<generator>MediaWiki 1.43.5</generator>
	<entry>
		<id>https://docs.moodle.org/310/en/index.php?title=Development:Weekly_Code_Review&amp;diff=80939</id>
		<title>Development:Weekly Code Review</title>
		<link rel="alternate" type="text/html" href="https://docs.moodle.org/310/en/index.php?title=Development:Weekly_Code_Review&amp;diff=80939"/>
		<updated>2011-01-29T14:36:35Z</updated>

		<summary type="html">&lt;p&gt;Alicethomas: &lt;/p&gt;
&lt;hr /&gt;
&lt;div&gt;Moodle has a weekly process of code review in the stable branches to help improve the quality of stable packages and to help detect any regressions that may have occurred in stable code.&lt;br /&gt;
&lt;br /&gt;
Here is the process for testers and developers:&lt;br /&gt;
&lt;br /&gt;
==Every Tuesday==&lt;br /&gt;
&lt;br /&gt;
===Don&#039;t check any new fixes into CVS===&lt;br /&gt;
&lt;br /&gt;
HEAD is OK, but please avoid new fixes in all STABLE branches unless they are definitely definitely not going to cause new bugs  ;-)&lt;br /&gt;
&lt;br /&gt;
===Find bugs to test===&lt;br /&gt;
&lt;br /&gt;
All developers and testers should be looking at:&lt;br /&gt;
&lt;br /&gt;
 Moodle 1.9 stable:  [http://tracker.moodle.org/secure/IssueNavigator.jspa?mode=hide&amp;amp;requestId=10565 1.9.x Needs QA review]&lt;br /&gt;
&lt;br /&gt;
(Make sure you can see the QA Assigned column)&lt;br /&gt;
&lt;br /&gt;
Grab any bug that:&lt;br /&gt;
&lt;br /&gt;
* Is from the last week&lt;br /&gt;
* You did not fix&lt;br /&gt;
* Does not have a QA assignee yet &lt;br /&gt;
&lt;br /&gt;
and claim it by editing the bug to add your name as QA Assignee.&lt;br /&gt;
&lt;br /&gt;
===Test the fix===&lt;br /&gt;
&lt;br /&gt;
* Read the comments &lt;br /&gt;
* Look at the patch attached to the tracker&lt;br /&gt;
* Try the feature out to make sure it works as advertised &lt;br /&gt;
** Ensure you have debugging set to the DEBUG_DEVELOPER &amp;amp; look for new errors&lt;br /&gt;
** Test for potential regressions in other areas the fix may affect&lt;br /&gt;
* Post new comments (or talk on Moodle HQ) if you need clarification&lt;br /&gt;
&lt;br /&gt;
===Deal with any problems===&lt;br /&gt;
&lt;br /&gt;
* Post a comment on the bug with details (preferably including patch!)&lt;br /&gt;
* Reopen the bug&lt;br /&gt;
* If it looks like the original developer isn&#039;t seeing it, reassign to Eloy Lafuente for [[Development:Bug triage|triage]]&lt;br /&gt;
* If the fix is obvious and you have CVS access, fix it yourself&lt;br /&gt;
&lt;br /&gt;
===Mark the fix as reviewed===&lt;br /&gt;
&lt;br /&gt;
* When you are satisfied the bug is correctly fixed, simply change the status from Resolved to Closed&lt;br /&gt;
* If you committed any fix, let somebody else review it again - do not Close it yourself&lt;br /&gt;
&lt;br /&gt;
==Every Wednesday==&lt;br /&gt;
&lt;br /&gt;
* The download site will automatically tag the whole current stable codebase (e.g. MOODLE_19_WEEKLY) at 1:00 GMT on Wednesday morning and then build download packages&lt;br /&gt;
&lt;br /&gt;
==See also==&lt;br /&gt;
&lt;br /&gt;
*Using Moodle [http://moodle.org/mod/forum/discuss.php?d=93474 Tuesday Code Reviews] forum discussion&lt;br /&gt;
&lt;br /&gt;
[[Category:Developer|Weekly Code Review]]&lt;br /&gt;
[[Category:Quality Assurance]]&lt;br /&gt;
* [http://www.customwritinghelp.co.uk/assignment.php assignment writing]&lt;br /&gt;
* [http://www.perfectwriting.co.uk/assignment/assignment-help.php assignment help]&lt;/div&gt;</summary>
		<author><name>Alicethomas</name></author>
	</entry>
</feed>