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

Changes needed to compile scidoc v7 updates #10

Merged
merged 2 commits into from
Aug 27, 2024

Conversation

grantfirl
Copy link

No description provided.

@grantfirl grantfirl requested a review from mzhangw as a code owner August 27, 2024 18:22
@@ -63,6 +63,7 @@ module ugwpv1_gsldrag
!>@brief The subroutine initializes the unified UGWP
!> \section arg_table_ugwpv1_gsldrag_init Argument Table
!! \htmlinclude ugwpv1_gsldrag_init.html
!!
Copy link
Author

Choose a reason for hiding this comment

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

@mzhangw We need to keep an empty line with !! after the htmlinclude line in order for ccpp_prebuild to properly parse the file.

@@ -2,6 +2,10 @@
!!
module machine

!! \section arg_table_machine
Copy link
Author

Choose a reason for hiding this comment

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

@mzhangw Whenever a type (DDT) is defined that is required for the CCPP, it must have this same Doxygen markup in order for ccpp_prebuild to find it.

Copy link
Owner

@mzhangw mzhangw left a comment

Choose a reason for hiding this comment

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

Looks good to me

@mzhangw mzhangw merged commit 8442afa into mzhangw:scidoc_v7 Aug 27, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants