Skip to content

DOC: Consolidating doxygen config files#5313

Merged
dzenanz merged 1 commit intoInsightSoftwareConsortium:masterfrom
albert-github:feature/issue_5295
Apr 22, 2025
Merged

DOC: Consolidating doxygen config files#5313
dzenanz merged 1 commit intoInsightSoftwareConsortium:masterfrom
albert-github:feature/issue_5295

Conversation

@albert-github
Copy link
Copy Markdown
Contributor

@albert-github albert-github commented Apr 17, 2025

  • Use in the wrapping process the same configuration file for the documentation as in case for the normal documentation.
  • remove the now obsolete wrapping doxygen configuration file

All based on, and closes #5295.

PR Checklist

Copy link
Copy Markdown
Member

@dzenanz dzenanz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but Matt and/or Hans should review.

@dzenanz dzenanz requested review from hjmjohnson and thewtex April 17, 2025 13:35
Copy link
Copy Markdown
Member

@thewtex thewtex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@albert-github thank you!!

Please see the inline comment.

Comment thread Wrapping/macro_files/itk_end_wrap_module.cmake Outdated
Copy link
Copy Markdown
Member

@dzenanz dzenanz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be squashed upon merge.

- Use in the wrapping process the same configuration file for the documentation as in case for the normal documentation.
- remove the now obsolete wrapping doxygen configuration file
@dzenanz dzenanz force-pushed the feature/issue_5295 branch from d45121e to ed5250a Compare April 21, 2025 21:18
@dzenanz dzenanz requested a review from thewtex April 21, 2025 21:18
@dzenanz
Copy link
Copy Markdown
Member

dzenanz commented Apr 21, 2025

I squashed the changes into one commit.

Copy link
Copy Markdown
Member

@thewtex thewtex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

@dzenanz dzenanz merged commit 8657a1c into InsightSoftwareConsortium:master Apr 22, 2025
16 checks passed
@dzenanz
Copy link
Copy Markdown
Member

dzenanz commented Apr 22, 2025

Thank you for tackling this Albert!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Python wrapping Python bindings for a class type:Documentation Documentation improvement or change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consolidating the config files from the main ITK directory and the wrapping directory

3 participants