<div dir="ltr">Folks<div><br></div><div>I wanted to raise awareness about one of the things I captured while doing reviews recently - we are sacrificing quality to bugfixing and feature development velocity, essentially moving from one heap to another - from bugs/features to 'tech-debt' bugs.</div><div><br></div><div>I understand that we all have deadlines and need to meet them. But, folks, let's create the following policy:</div><div><br></div><div>1) do not introduce hacks/workarounds/kludges if it is possible.</div><div>2) while fixing things if you have a hack/workaround/kludge that you need to work with - think of removing it instead of enhancing and extending it. If it is possible - fix it. Do not let our technical debt grow.</div><div>3) if there is no way to avoid kludge addition/enhancing, if there is no way to remove it - please, add a 'TODO/FIXME' line above it, so that we can collect them in the future and fix them gradually.</div><div><br></div><div>I suggest to add this requirement into code-review policy.</div><div><br></div><div>What do you think about this?<br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div><div dir="ltr">Yours Faithfully,<br>Vladimir Kuklin,<br>Fuel Library Tech Lead,<br>Mirantis, Inc.<br>+7 (495) 640-49-04<br>+7 (926) 702-39-68<br>Skype kuklinvv<br>35bk3, Vorontsovskaya Str.<br>Moscow, Russia,<br><a href="http://www.mirantis.ru/" target="_blank">www.mirantis.com</a><br><a href="http://www.mirantis.ru/" target="_blank">www.mirantis.ru</a><br><a href="mailto:vkuklin@mirantis.com" target="_blank">vkuklin@mirantis.com</a></div></div></div></div>
</div></div>