<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D10151" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Thanks for the patch! In general, it's best to title your Phabricator revision with a short description of the change or fix itself, and mention the bug in the Summary section. And when you do so, you'll want to put it in the form of "BUG: 382512" on its own line; this will cause that bugzilla ticket to automatically get closed if and when this lands. The Summary section should also receive a longer explanation of the change, if necessary. Also, please write something in the Test Plan section describing how you tested this change. Thanks again for the patch, and don't hesitate to ask if anything's unclear!</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R106 KSysguard</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10151" rel="noreferrer">https://phabricator.kde.org/D10151</a></div></div><br /><div><strong>To: </strong>carlavilla, mlaurent, ngraham<br /><strong>Cc: </strong>ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>