Skip to content

Commit

Permalink
Use cppcheck to scan the C++ code
Browse files Browse the repository at this point in the history
Fix #379
  • Loading branch information
Minggang Wang committed Jul 19, 2018
1 parent 7549cfe commit 9ea3896
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,4 @@ before_install:
- sudo docker build -t rcldocker .

script:
- sudo docker run -v $(pwd):/root/rclnodejs --rm rcldocker bash -i -c 'cd /root/rclnodejs && ./scripts/build.sh && npm test'
- sudo docker run -v $(pwd):/root/rclnodejs --rm rcldocker bash -i -c 'cd /root/rclnodejs && cppcheck --enable=all src/*.cpp src/*.hpp && ./scripts/build.sh && npm test'
3 changes: 2 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,8 @@ RUN apt update && apt install -y \
python3-setuptools \
python3-yaml \
python3-colcon-common-extensions \
cmake
cmake \
cppcheck

RUN pip3 install -U setuptools

Expand Down
2 changes: 1 addition & 1 deletion circle.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ dependencies:
pre:
- brew update
- brew uninstall python
- brew install https://raw.githubusercontent.com/Homebrew/homebrew-core/f2a764ef944b1080be64bd88dca9a1d80130c558/Formula/python.rb
- brew install python
- brew install python@2
- brew install wget cmake cppcheck tinyxml eigen pcre poco
# install dependencies for Fast-RTPS if you are using it
Expand Down

0 comments on commit 9ea3896

Please sign in to comment.