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

Fixing documentation issues #1161

Merged
merged 15 commits into from May 9, 2016
Merged

Fixing documentation issues #1161

merged 15 commits into from May 9, 2016

Conversation

dcoeurjo
Copy link
Member

@dcoeurjo dcoeurjo commented Apr 23, 2016

PR Description

This PR contains documentation fixes (starting with some geometry module pages). I'll update this PR, I'm just creating it to keep you in touch.

Checklist

  • Unit-test of your feature with Catch.
  • Doxygen documentation of the code completed (classes, methods, types, members...)
  • Documentation module page added or updated.
  • New entry in the ChangeLog.md added.
  • No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).
  • All continuous integration tests pass (Travis & appveyor)

@dcoeurjo
Copy link
Member Author

dcoeurjo commented May 9, 2016

can someone check this one?

@jlevallois
Copy link
Member

Seems good for me (except for the #pragma GCC diagnostic ignored "-Wdeprecated-declarations" part because I don't know why you need this).

:octocat:

@dcoeurjo
Copy link
Member Author

dcoeurjo commented May 9, 2016

some GCC/CLANG compiler complain due to warnings in ITK/Eigen which are not related to DGtal.

Thanks, merging,

@dcoeurjo dcoeurjo merged commit decf390 into DGtal-team:master May 9, 2016
@dcoeurjo dcoeurjo deleted the docFix branch October 7, 2021 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants