Skip to content

Commit

Permalink
Add test that runs pep8 on vcsvtk for syntax checking.
Browse files Browse the repository at this point in the history
Fixes #1244.

This should eventually be expanded to the rest of vcs as the authors
find time to update their code.
  • Loading branch information
David C. Lonie authored and doutriaux1 committed Jun 15, 2015
1 parent 3219bba commit 04d8c2b
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions testing/vcs/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -671,5 +671,15 @@ cdat_add_test(vcs_test_colorpicker_appearance
${cdat_SOURCE_DIR}/testing/vcs/test_vcs_colorpicker_appearance.py
${BASELINE_DIR}/test_vcs_colorpicker_appearance.png
)

# Run pep8 to check code style:
add_test(vcs_pep8
"${CMAKE_INSTALL_PREFIX}/bin/pep8"
# --count is needed so that the exit code is nonzero when errors are found.
--count
# Add more file/dir paths or modify this one to expand the coverage:
"${cdat_SOURCE_DIR}/Packages/vcs/Lib/vcsvtk/"
)

add_subdirectory(vtk_ui)
add_subdirectory(editors)

0 comments on commit 04d8c2b

Please sign in to comment.