<div dir="ltr"><br><div class="gmail_quote"><div dir="ltr"><div>Hello folks,</div><div><br></div><div>A while ago we talked about using clang-format for our codebase[1]. We started doing several pieces of this work asynchronously. Here&#39;s an update on the current state of affairs:<br></div><div><br></div><div>* Team agrees on a style and a config file representing the style.</div><div></div><div>This has been happening asynchronously on Github[2]. Amar, Xavi, and Jeff -- Can we close out this discussion and have a config file in 2 weeks? If anyone feels strongly about coding style, please participate in the discussion now.<br></div><br><div>* Commit the coding style guide to codebase and make changes in rfc.sh to use it.</div><div></div><div>Waiting on 1. I can do this once we have the discussion finalized.<br></div><div><br></div><div>* gluster-ant commits a single large patch for whole codebase with a standard clang-format style.<br></div><div></div><div>This is waiting on the first two steps and should be trivial to accomplish. I have access to the gluster-ant account and I can make the necessary changes.<br></div><div><br> </div><div>* Have the job ready to check the patch with the config file, on the server side, this should be a Voting job in smoke.</div><div>The server side Jenkins job is now ready[3]. The client side rfc.sh patch is next, but merging that change will wait on the config file being ready.</div><div><br></div><div>[1]: <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1564149#c33" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=1564149#c33</a></div><div>[2]: <a href="https://github.com/nigelbabu/clang-format-sample/" target="_blank">https://github.com/nigelbabu/clang-format-sample/</a></div><div>[3]: <a href="https://review.gluster.org/#/c/20418/" target="_blank">https://review.gluster.org/#/c/20418/</a><br></div><br clear="all"></div></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">nigelb<br></div></div></div>