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

Add doxygen to the build #1171

Open
edwardhartnett opened this issue Jan 12, 2024 · 4 comments
Open

Add doxygen to the build #1171

edwardhartnett opened this issue Jan 12, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@edwardhartnett
Copy link
Contributor

Is your feature request related to a problem? Please describe.
With other EMC packages, we have an ENABLE_DOCS cmake option, which causes the documentation to be built on make all.

This has several advantages, including ease of checking documentation build in CI and the ability to inject the software version into the documentation, which is important.

Describe the solution you'd like
Add ENABLE_DOCS option to build, and test the documentation build in CI.

Describe alternatives you've considered
When documenting UFS_UTILS, NCEPLIBS, UPP, and soon fv3atm, we have worked out a good way to do this which is easy for programmers.

Additional context
Once we are building the documentation, we can use the gh-pages branch to display it on GitHub.

@edwardhartnett edwardhartnett added the enhancement New feature or request label Jan 12, 2024
@MatthewMasarik-NOAA
Copy link
Collaborator

@edwardhartnett, I do have a PR for this capability in the works.

@edwardhartnett
Copy link
Contributor Author

Oh, I have one that works that I was about to put up. It matches the way we do things in the other EMC repos to build doxygen.

It also causes the documentation to be built in CI, and then uploads a copy of the built documentation to GitHub actions, so programmers can download it, in case that is of interest.

We could also arrange to put the develop branch documentation automatically on the gh-pages branch. Typically we use the gh-pages branch for the latest released version of the doxygen docs, but I don't think you have one yet.

@MatthewMasarik-NOAA
Copy link
Collaborator

MatthewMasarik-NOAA commented Jan 12, 2024 via email

@edwardhartnett
Copy link
Contributor Author

We would like all the EMC repos to do this the same way, so let's make sure that we end up with something that looks and works the same way as NCEPLIBS, UPP, UFS_UTILS, and (soon) fv3atm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants