VixenEngine icon indicating copy to clipboard operation
VixenEngine copied to clipboard

[All] documentation comments need to be added for doxygen

Open MattGuerrette opened this issue 9 years ago • 7 comments

Vixen needs to have its code documented. This way, we can generate doxygen documentation that can eventually be linked to on developer site.

MattGuerrette avatar Oct 03 '15 07:10 MattGuerrette

Shall I start the process of documenting? I haven't much experience with DoxyGen, but it shouldn't be too difficult.

GonzoBright avatar Oct 04 '15 18:10 GonzoBright

If you would like to sure. I would first research the format for doxygen comments on their website.

MattGuerrette avatar Oct 04 '15 19:10 MattGuerrette

As a matter of fact I'm doing that right now. Are you planning to add contributors to the project, or shall the project be solely based on pull requests?

GonzoBright avatar Oct 04 '15 19:10 GonzoBright

at the moment, it will be based on pull requests. Those that will be added as collaborators will be accounts that have made multiple pull requests and have contributed most to the project. That way I don't add a bunch of collaborators who aren't actually doing anything for the project.

MattGuerrette avatar Oct 04 '15 19:10 MattGuerrette

Perfect. That'll do.

I've noticed there's already a directory with all of the applicable doxygen files, did you perform a complete setup beyond the insturction to make the tool itself? In essence, I'm trying to get an idea as to where to start.

GonzoBright avatar Oct 04 '15 19:10 GonzoBright

The existing files in the doxygen folder can be removed. That is an old version. They will most likely be written over anyway, if you output to that folder. But to be safe, delete all files in that directory and then run doxygen

MattGuerrette avatar Oct 04 '15 19:10 MattGuerrette

Note: Doxygen should only be run on source and include files actually being built into the engine. So, all files in UNUSED folders should be added to the excluded list when running doxygen.

MattGuerrette avatar Oct 04 '15 19:10 MattGuerrette