A collection of updates for the addons template.#14
A collection of updates for the addons template.#14arturoc merged 12 commits intoopenframeworks:masterfrom bakercp:master
Conversation
|
@bilderbuchi - I just merged the changes from @LeoColomb. Sorry @LeoColomb I didn't see your existing PR in here. |
|
I just had time for a quick once-over, but this looks good as far as I can see. Only thing, about that |
|
Oh I just see you add a empty
|
|
@LeoColomb @bilderbuchi Yes, I agree -- an empty tests folder with no documentation is counter productive. Before merging this I'll delete it. I added it because of a recent set of discussions around unit testing, but since that discussion isn't finished, a tests folder doesn't make sense for now. @LeoColomb - do you feel the same way about the docs folder? It seems that if everyone fills out a simple Doxygen configuration file, the ecosystem of oF documentation would be greatly improved ... as it sets up an expectation that code should be at least minimally documented :) |
|
Well yes, I do. Furthermore, docs isn't the right name for a folder containing Doxygen configuration file, because as explained in GitHub Guides, |
|
Ping? |
|
thanks! |
See the commit log for more specifics.