jamoma / Jamoma

Umbrella repository for the Jamoma platform
http://jamoma.org/
Other
33 stars 3 forks source link

StyleGuide needs a section on comments #122

Closed nwolek closed 9 years ago

nwolek commented 10 years ago

This file: https://github.com/jamoma/Jamoma/blob/dev/Doxygen/intro_chapters/C-StyleGuide.dox

Shows up on the web here: http://api.jamoma.org/appendix_styleguide.html

We should mention:

lossius commented 10 years ago

Hi Nathan,

I did some work on this a few days ago, and you can see the result here:

http://api.jamoma.org/appendix_styleguide.html

Could you please check if this is good enough for now, or eventually add stuff that you're missing? I believe this issue is mostly done and almost ready to be closed.

Thanks, Trond

nwolek commented 10 years ago

A great addition. Thanks, Trond!

1) Re your guidance on brief: "\@brief - single sentence typically starting by the module name"

I thought we agreed in Bergen that starting with the module name was redundant. I would rather they start with a verb.

2) It would be good to embed real code examples into this chapter via the @snippet tag. You can see that I did this in the 60-UnitTesting.dox chapter. This way when the actual code is updated, it is automatically updated in the Doxygen chapter.

If you want me to handle this, you can make a ticket and assign to me.

-Nathan

On May 8, 2014, at 3:22 PM, Trond Lossuis notifications@github.com wrote:

Hi Nathan,

I did some work on this a few days ago, and you can see the result here:

http://api.jamoma.org/appendix_styleguide.html

Could you please check if this is good enough for now, or eventually add stuff that you're missing? I believe this issue is mostly done and almost ready to be closed.

Thanks, Trond

— Reply to this email directly or view it on GitHub.