search

NOAA-EMC / UPP

Other
33 stars 95 forks source link

turn on doxygen build in CI #932

Closed edwardhartnett closed 2 months ago

edwardhartnett commented 3 months ago

Fixes #847

This will build documentation in the CI, but will not fail CI for any documentation warnings.

However, this will causes the documentation warnings to appear in the build output, and developers can target them clear them up.

Once they are all gone, we can turn on WARN_AS_ERROR in doxygen, and ensure that new code always passes documentation check before being merged.

WenMeng-NOAA commented 3 months ago

@edwardhartnett Please sync your branch with the latest UPP develop.

edwardhartnett commented 3 months ago

Done.

WenMeng-NOAA commented 2 months ago

@FernandoAndrade-NOAA Please run the testing from your end.

FernandoAndrade-NOAA commented 2 months ago

@FernandoAndrade-NOAA Please run the testing from your end.

No changes from my side of RTs on Hera, Hercules, and Orion.

WenMeng-NOAA commented 2 months ago

The UPP RTs were completed on WCOSS2 without baseline changes. This PR is ready for merging.