<div dir="ltr"><div>Hi everybody,</div><div><br></div>It is not easy for new comers to know about conventions and best practices to be followed when contributing to Subsurface. This creates overhead for new comers and code maintainers.<div>New comers have to extensively browse the code to find utility functions (or end up re-implement things that already exist), figure what should be done where, etc... and often end up having to re-implement fixes since original pull request do not follow Subsurface development common practices.</div><div>Code maintainers spend lot of time (re)explaining such practices while doing code reviews.</div><div><br></div><div>To cope with this I've prepared a change introducing a new CONVENTIONS.md file that is meant to collect such information: <a href="https://github.com/Subsurface-divelog/subsurface/pull/1196">https://github.com/Subsurface-divelog/subsurface/pull/1196</a> Any feedback is welcome. Since it is the one convention I learned during a recent pull request I proposed, I started with documenting the use of membuffer for string manipulations...</div><div><br></div><div>I know that the majority of frequent contributors (that are the ones knowing such conventions) do not have much free time to spend on describing them. On my side I'm still pretty fresh with Subsurface development and do not yet know much of them, but do currently have some free time to spend on this topic. So feel free to post me a small note in this thread with some of the conventions you know about, I'll happily prepare pull request with detailed descriptions and examples.</div><div><br></div><div>While I'm talking about documentation for contributors, I have a question regarding Subsurface website, more specifically the <a href="https://subsurface-divelog.org/documentation/contributing/">https://subsurface-divelog.org/documentation/contributing/</a> page.</div><div>I've made a (already merged) change with a mention to the usage of CHANGELOG.md <a href="https://github.com/Subsurface-divelog/subsurface/pull/1194">https://github.com/Subsurface-divelog/subsurface/pull/1194</a></div><div>I have another one (to be reviewed) fixing few spelling mistakes: <a href="https://github.com/Subsurface-divelog/subsurface/pull/1195">https://github.com/Subsurface-divelog/subsurface/pull/1195</a></div><div>Then if/when the currently discussed pull request would be approved it would be nice to reflect these changes in the corresponding page on <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"><a href="https://subsurface-divelog.org">https://subsurface-divelog.org</a>. The question is: Was this page manually edited in Subsurface-website repository or was it actually generated from markdown? If so, is the script available somewhere?</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>Wish you all a great day,</div><div><br></div><div>Jérémie</div></div>