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.

Sign up for GitHub

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

Jump to bottom

CI: documentation checks fail on github actions #301

Closed
dschlaep opened this issue Jan 24, 2022 · 0 comments
Closed

CI: documentation checks fail on github actions #301

dschlaep opened this issue Jan 24, 2022 · 0 comments
Assignees
@N1ckP3rsl3y
Labels
closed_and_not_yet_released
Milestone
Code testing

Comments

@dschlaep
Copy link
Member

Step "Build documentation and check for warnings" fails (last confirmed on Jan 24, 2022 for commit bd26dc276ba549f5b691bb065f65a27b7a12e4d4) with

make: *** [makefile:293: doc] Error 1
Error: Process completed with exit code 2.

https://github.com/DrylandEcology/SOILWAT2/runs/4925554260?check_suite_focus=true

no other information is provided

doxygen v1.8.17 is installed

Creating doxygen documentation works without warnings/errors on my local setup (with doxygen v1.9.3)

-> enable this check under branch protection once this issue is resolved
@dschlaep dschlaep added this to the Code testing milestone Jan 24, 2022
dschlaep added a commit that referenced this issue Jan 24, 2022
@dschlaep
Merge pull request #297 from DrylandEcology/feature_ghactions_v2
c1638ac 
- move CI from travis and appveyor to github actions (close #298 but see #300 and #301)
- updated googletest (as of v1.11.0, Jun 11, 2021) requires newer gcc and clang versions (>= 5.0, https://github.com/google/googletest) 
  than currently implemented for travis and appveyor scripts (gcc v4.9 and clang v3.5)
- avoid codecov's bash uploader; it will be retired by Feb 1, 2022 (https://docs.codecov.com/docs/about-the-codecov-bash-uploader)
- fix compiler "Warning: array subscript has type char" for function `SW_MDL_read()`
- update documentation (close #267 but see #299)
@dschlaep dschlaep mentioned this issue Nov 30, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@N1ckP3rsl3y N1ckP3rsl3y

Labels
closed_and_not_yet_released
Projects
None yet
Milestone
Code testing
Development

No branches or pull requests
2 participants
@dschlaep @N1ckP3rsl3y