<div dir="ltr">Hey All,<div><br></div><div><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">While trying to review an upstream patch, I noticed that a single function has almost grown to 1k LOC :-). I found it unwieldy from a readability perspective and problems of that nature could be a major deterrent for anybody to contribute code in such areas. I am inclined to believe that this is not a one-off and that we have areas in our modules where code readability could be significantly improved.</span></div><div><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline"><br></span></div><div>Code readability is a form of technical debt and we have some more technical debt already identified at [1].  While accruing technical debt is easy and common, it also requires a dedicated effort to retire that. <span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">If you are already following some best practices for your component to address technical debt, please share them here for the benefit of everyone! :-). </span> </div><div><br></div><div>I am also wondering if we could institute a process for addressing common technical debt (test coverage, readability/refactor, long pending problems, outstanding bugs etc.) release over release. </div><div>Appreciate your thoughts that can help in reducing debt and make it easier for all of us to take the project further!<br></div><div><br></div><div>Thanks,</div><div>Vijay</div><div><br></div><div>[1] <a href="https://github.com/gluster/glusterfs/labels/FA%3A%20Technical%20Debt">https://github.com/gluster/glusterfs/labels/FA%3A%20Technical%20Debt</a></div><div><br class="gmail-Apple-interchange-newline"><br></div></div>