<table><tr><td style="">ngraham added reviewers: broulik, Plasma, ngraham.<br />ngraham added a comment.
</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/D20181">View Revision</a></tr></table><br /><div><div><p>Thanks very much for this, it looks great! This will land in 5.58 not 5.57 since we don't typically commit patches with string changes in the last two weeks before frameworks tagging (which happens in five days, see <a href="https://community.kde.org/Schedules/Frameworks" class="remarkup-link" target="_blank" rel="noreferrer">https://community.kde.org/Schedules/Frameworks</a>). So please change all the <tt style="background: #ebebeb; font-size: 13px;">@since</tt> tags. Thanks!</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R292 KUnitConversion</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20181">https://phabricator.kde.org/D20181</a></div></div><br /><div><strong>To: </strong>JJRcop, broulik, Plasma, ngraham<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>