widelands-dev team mailing list archive

[SirVer <SirVer@xxxxxx>]

SirVer has proposed merging lp:~widelands-dev/widelands/codecheck into lp:widelands.

Commit message:
Enable building documentation with nit-picky warnings as errors on travis.
Enable running running the test suite of our lints on travis.

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/codecheck/+merge/326647
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/codecheck into lp:widelands.

=== modified file '.travis.sh'
--- .travis.sh	2017-06-27 16:45:47 +0000
+++ .travis.sh	2017-07-02 20:44:46 +0000
@@ -30,6 +30,7 @@
    libglew-dev \
    libicu-dev \
    libpng-dev \
+   python-sphinx \
    libsdl2-dev \
    libsdl2-image-dev \
    libsdl2-mixer-dev \
@@ -42,8 +43,17 @@
 cd build
 cmake .. -DCMAKE_BUILD_TYPE:STRING="$BUILD_TYPE"
 
-# Any codecheck warning is an error in Debug builds. Keep the codebase clean!!
 if [ "$BUILD_TYPE" == "Debug" ]; then
+   # Build the documentation. Any warning is an error.
+   ./extract_rst.py
+   sphinx-build -n -W -b json -d build/doctrees source build/json
+
+   # Run the codecheck test suite.
+   pushd cmake/codecheck
+   ./run_tests.py
+   popd
+
+   # Any codecheck warning is an error in Debug builds. Keep the codebase clean!!
    # Suppress color output.
    TERM=dumb make -j1 codecheck 2>&1 | tee codecheck.out
    if grep '^[/_.a-zA-Z]\+:[0-9]\+:' codecheck.out; then