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: io/module_wrt_grid_comp.F90 and io/post_fv3.F90 and io/post_nems_routines.F90 #769

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from

Conversation

edwardhartnett
Copy link
Contributor

@edwardhartnett edwardhartnett commented Jan 19, 2024

Description

More doxygen. This PR is ready to merge.

Issue(s) addressed

Part of #760

Testing

Documentation changes only.

Dependencies

N/A

@edwardhartnett edwardhartnett marked this pull request as draft January 19, 2024 15:02
@DusanJovic-NOAA DusanJovic-NOAA changed the title more doxygen: io/module_write_internal_state.F90 and io/module_wrt_grid_comp.F90 Doxygen: io/module_write_internal_state.F90 and io/module_wrt_grid_comp.F90 Jan 26, 2024
@edwardhartnett edwardhartnett changed the title Doxygen: io/module_write_internal_state.F90 and io/module_wrt_grid_comp.F90 Doxygen: io/module_wrt_grid_comp.F90 Jan 28, 2024
@edwardhartnett
Copy link
Contributor Author

@junwang-noaa and @DusanJovic-NOAA this is the next branch that needs documentation added...

@edwardhartnett edwardhartnett marked this pull request as ready for review February 21, 2024 11:34
@edwardhartnett
Copy link
Contributor Author

@junwang-noaa as requested I combined the two open documentation PRs into this one.

I believe this is ready to merge.

@edwardhartnett edwardhartnett changed the title Doxygen: io/module_wrt_grid_comp.F90 Doxygen: io/module_wrt_grid_comp.F90 and io/post_fv3.F90 and io/post_nems_routines.F90 Feb 21, 2024
@edwardhartnett
Copy link
Contributor Author

Can we get this PR merged please?

@DusanJovic-NOAA
Copy link
Collaborator

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.

3 participants