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

setup of github actions for ci #297

Merged
merged 19 commits into from
Jan 24, 2022
Merged

setup of github actions for ci #297

merged 19 commits into from
Jan 24, 2022

Commits on Jan 23, 2022

  1. github actions: add g++ for cygwin

    - the development should have been on a dedicated branch instead of on master since the start ...!
    - add g++ for cygwin build
    dschlaep committed Jan 23, 2022
    Configuration menu
    Copy the full SHA
    09c0e5f View commit details
    Browse the repository at this point in the history
  2. Fix compiler "Warning: array subscript has type char"

    - warning was issued on github actions for a build on windows-release with cygwin and gcc v11.2.0
    
    - the argument of `isalpha()` is "int ch" (https://en.cppreference.com/w/c/string/byte/isalpha) but we were passing in a "char" (which may be signed or unsigned depending on platform and compiler)
    - similarly to the use in the next line with `toupper()`, the fix is to convert to "int"
    dschlaep committed Jan 23, 2022
    Configuration menu
    Copy the full SHA
    59e1486 View commit details
    Browse the repository at this point in the history

Commits on Jan 24, 2022

  1. github actions: no saniziters on cygwin

    AddressSanitizer and LeakSanitizer are not available on cygwin (https://github.com/google/sanitizers, but see https://github.com/google/sanitizers/wiki/AddressSanitizerWindowsPort)
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    968e7e6 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    ec65df4 View commit details
    Browse the repository at this point in the history
  3. github actions: check documentation

    - new step "check_documentation" (only on linux): install doxygen, build SOILWAT2 documentation, and check that no unexpected warnings occurred
    
    - also: re-organized code coverage checks as separate job
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    b043330 View commit details
    Browse the repository at this point in the history
  4. github actions: fix separate jobs

    - each job needs to check out the repository
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    a5b2154 View commit details
    Browse the repository at this point in the history
  5. github actions: re-organize yaml scripts

    - remove "main.yml"
    - move job "build_nix" into script "main_nix.yml"
    - move job "check_documentation" into script "check_doc.yml"
    - move job "build_Windows_Cygwin" into script "main_win.yml"
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    43f2ea4 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    1573f23 View commit details
    Browse the repository at this point in the history
  7. Update README

    - update compiler version requirements for unit tests
    - update status badges (moving from travis and appveyor to github actions)
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    a5b21e9 View commit details
    Browse the repository at this point in the history
  8. github actions: create check_doc.yml

    - move job "check_documentation" into script "check_doc.yml" (previously in "main.yml")
    - install also doxygen-latex and graphviz dependencies
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    86d91ff View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    f468bea View commit details
    Browse the repository at this point in the history
  10. github actions: use shorter name for doc workflow

    - workflow names are used in badges on README -> long names display poorly
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    9718a5a View commit details
    Browse the repository at this point in the history
  11. github actions: use shorter name for nix workflow

    - workflow names are used in badges on README -> long names display poorly
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    3949a70 View commit details
    Browse the repository at this point in the history
  12. github actions: use shorter name for win workflow

    - workflow names are used in badges on README -> long names display poorlygithub actions: use shorter name for doc workflow
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    59f03ef View commit details
    Browse the repository at this point in the history
  13. Configuration menu
    Copy the full SHA
    d214cb9 View commit details
    Browse the repository at this point in the history
  14. Configuration menu
    Copy the full SHA
    ec25aad View commit details
    Browse the repository at this point in the history
  15. Fix doxygen documentation "empty return type"

    - fix doxygen "warning: documented empty return type ..." for functions that return "void": `lyrSoil_to_lyrTemp()`, `pot_soil_evap_bs()`, `write_headers_to_csv()`
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    606de72 View commit details
    Browse the repository at this point in the history
  16. Update documentation

    - some doxygen hyperlinks no longer worked correctly -> use mostly markdown-style links (except for creating subpages)
    - however, html tags don't work for doxygen documentation (as of v1.9.3) (see issue #299)
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    b562710 View commit details
    Browse the repository at this point in the history
  17. Remove obsolete doxygen exception

    - close issue #267 (Remove from doc/doxygen_exceptions.txt: "Unexpected html tag <img>...")
    - doxygen issue resolved with v1.8.16 (github actions installs v1.8.17 on ubuntu-release; and I get v1.9.3)
    dschlaep committed Jan 24, 2022
    Configuration menu
    Copy the full SHA
    3ecbdc9 View commit details
    Browse the repository at this point in the history