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

hoge #33

Closed
wants to merge 16 commits into from
11 changes: 7 additions & 4 deletions .github/workflows/cpp-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,23 +9,26 @@ jobs:

steps:
- name: Checkout code
uses: actions/checkout@v1
uses: actions/checkout@v2

- name: Install Cppcheck
run: sudo apt-get install -y cppcheck
run: |
sudo apt-get update
sudo apt-get install -y cppcheck

- name: Get changed files
id: changed-files
run: |
git fetch origin ${{ github.base_ref }} --depth=1
git diff --name-only origin/${{ github.base_ref }} ${{ github.head_ref }} > changed_files.txt
git diff --name-only FETCH_HEAD ${{ github.sha }} > changed_files.txt
cat changed_files.txt

- name: Run Cppcheck on changed files
run: |
files=$(cat changed_files.txt | grep '\.cpp$\|\.h$' | tr '\n' ' ')
if [ -n "$files" ]; then
cppcheck --enable=warning,style,performance --error-exitcode=1 $files
cppcheck --enable=warning,style,performance --error-exitcode=1 --xml $files 2> cppcheck-report.xml
else
echo "No C++ files changed."
fi
shell: bash
2 changes: 2 additions & 0 deletions perception/image_projection_based_fusion/src/debugger.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@
#include <cv_bridge/cv_bridge.h>
#endif

int hoge = 1;

Check warning on line 23 in perception/image_projection_based_fusion/src/debugger.cpp

View workflow job for this annotation

GitHub Actions / spell-check-partial

Unknown word (hoge)
void(hoge);

Check warning on line 24 in perception/image_projection_based_fusion/src/debugger.cpp

View workflow job for this annotation

GitHub Actions / spell-check-partial

Unknown word (hoge)
namespace
{

Expand Down
3 changes: 3 additions & 0 deletions perception/lidar_centerpoint/lib/centerpoint_trt.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,9 @@
// host
points_.resize(CAPACITY_POINT * config_.point_feature_size_);

int hoge = 2;

Check warning on line 89 in perception/lidar_centerpoint/lib/centerpoint_trt.cpp

View workflow job for this annotation

GitHub Actions / spell-check-partial

Unknown word (hoge)
void(hoge);

Check warning on line 90 in perception/lidar_centerpoint/lib/centerpoint_trt.cpp

View workflow job for this annotation

GitHub Actions / spell-check-partial

Unknown word (hoge)

// device
voxels_d_ = cuda::make_unique<float[]>(voxels_size_);
coordinates_d_ = cuda::make_unique<int[]>(coordinates_size_);
Expand Down
Loading