DOC: Eliminate obsolete Doxygen grouping #7421
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit removes the outdated Doxygen grouping, initially implemented to organize classes related to specific loadable modules. Since there were no systematic ways of checking that each classes were effectively associated with the correct group, the grouping was either incorrect or incomplete. This removes all doxygen commands starting with either `/ingroup Slicer_QtModules` or `/ingroup SlicerRt_QtModules`.
This commit removes the outdated Doxygen grouping, initially implemented to organize classes related to either MRML nodes or MRML widgets. Since there were no systematic ways of checking that each classes were effectively associated with the correct group, the grouping was either incorrect or incomplete. This removes all doxygen commands matching the following: * `/ingroup Slicer_MRMLWidgets` * /ingroup Slicer_MRML_Core`
This commit removes the outdated Doxygen grouping, initially implemented to organize classes related to Segmentation capabilities. Since there were no systematic ways of checking that each classes were effectively associated with the correct group, the grouping was either incorrect or incomplete. This removes all doxygen commands matching the following: * `/ingroup SegmentationCore` * `/ingroup Segmentations` * `/ingroup vtkSegmentationCore`
This commit removes the outdated Doxygen grouping, initially implemented to organize classes related to Slicer-specific CMake modules. Since there were no systematic ways of checking that each modules were effectively associated with the correct group, the grouping was either incorrect or incomplete. This removes all doxygen commands matching `/ingroup CMakeUtilities`
After copying and adapting the ITK classes based on the specific requirements of either the vtkITK or ResampleDTIVolume modules, the grouping was not relevant anymore.
|
Since these changes doesn't functionally impact the application and local compilation was successful, I will move forward with integration in the next hour. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This removes the outdated Doxygen grouping, initially implemented to organize classes related to Slicer-specific modules or capabilities.
Since there were no systematic ways of checking that each modules were effectively associated with the correct group, the grouping was either incorrect or incomplete.
This removes all doxygen commands matching
/ingroupand/defgroup