Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Doxygen config in repo is out of date - update. #19

Closed
schmie opened this issue Jul 9, 2015 · 9 comments
Closed

Doxygen config in repo is out of date - update. #19

schmie opened this issue Jul 9, 2015 · 9 comments
Labels
Milestone

Comments

@schmie
Copy link
Contributor

schmie commented Jul 9, 2015

Reported by jamespjh on 11 Oct 2011 17:22 UTC
There's a doxyfile in the repo, but it looks like it's not maintained (recursion was turned off so it was only doing the top level.)

Anyone currently using this? I'm willing to maintain it if no one else wants to.

@schmie
Copy link
Contributor Author

schmie commented Jul 9, 2015

Modified by hywel on 13 Oct 2011 11:41 UTC

@schmie schmie added this to the 0.2 milestone Jul 9, 2015
@schmie
Copy link
Contributor Author

schmie commented Jul 9, 2015

Comment by miguel on 24 Nov 2011 11:50 UTC
Moving to 0.3

@schmie schmie removed this from the 0.2 milestone Jul 9, 2015
@schmie
Copy link
Contributor Author

schmie commented Jul 9, 2015

Comment by jamespjh on 5 Dec 2011 11:26 UTC
Have been asked to do proper doxygen for my stuff as part of Code Review, and I agree, so addressing this now.

@schmie schmie added this to the 0.2 milestone Jul 9, 2015
@schmie
Copy link
Contributor Author

schmie commented Jul 9, 2015

Comment by jamespjh on 5 Dec 2011 12:49 UTC
Have updated the config file -- appearance is now much nicer -- although it would be nicer still if we update the version of doxygen on Pauli -- (which would need us to update the main ubuntu distro of pauli.)

@schmie
Copy link
Contributor Author

schmie commented Jul 9, 2015

Comment by jamespjh on 5 Dec 2011 12:49 UTC
CR please.

http://pauli.chem.ucl.ac.uk/trac/hemelb/changeset/464e2362ec1bffa9580f8258320e72e762bb5f28

@schmie schmie assigned hcarver and unassigned jamespjh Jul 9, 2015
@schmie
Copy link
Contributor Author

schmie commented Jul 9, 2015

Comment by jamespjh on 5 Dec 2011 12:50 UTC
Results at http://pauli.chem.ucl.ac.uk/jenkins/job/HemeLB_documentation/doxygen/

@schmie
Copy link
Contributor Author

schmie commented Jul 9, 2015

Comment by hywel on 5 Dec 2011 13:31 UTC
Yup, looks nice.

Not sure about the decision to include private members, though I guess it's right that it's consistent with the decision about testing of private members.

If we're ignoring parmetis, do we also want to ignore xml and xdr? Or is that a TODO until we have a dependency system?

@schmie schmie assigned jamespjh and unassigned hcarver Jul 9, 2015
@schmie
Copy link
Contributor Author

schmie commented Jul 9, 2015

Comment by jamespjh on 5 Dec 2011 13:37 UTC
I think include private because this is doc for us to help with development, internally, not doc for external users.

Yes, other dependencies can be tidied up with the main dependency system.

@schmie
Copy link
Contributor Author

schmie commented Jul 9, 2015

Modified by jamespjh on 5 Dec 2011 13:37 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants