Skip to content

Commit

Permalink
add cpp-linter (DeepLink-org#88)
Browse files Browse the repository at this point in the history
* add cpp-linter
  • Loading branch information
yangbofun authored May 24, 2023
1 parent fbbe8ca commit 1f508de
Show file tree
Hide file tree
Showing 3 changed files with 32 additions and 1 deletion.
25 changes: 25 additions & 0 deletions .github/workflows/cpp-linter.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
name: cpp-linter

on: pull_request

jobs:
cpp-linter:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: cpp-linter/cpp-linter-action@v2
id: linter
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
style: file
tidy-checks: -*

- name: Fail fast?!
if: steps.linter.outputs.checks-failed > 0
run: |
echo "Some files failed the linting checks!"
exit 1
# for actual deployment
# run: exit 1
6 changes: 6 additions & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
repos:
- repo: https://github.com/cpp-linter/cpp-linter-hooks
rev: v0.2.1 # Use the ref you want to point at
hooks:
- id: clang-format
args: [--style=file] # Other coding style: LLVM, GNU, Chromium, Microsoft, Mozilla, WebKit.
2 changes: 1 addition & 1 deletion DIOPI-IMPL/camb/error.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@
#include <cnrt.h>
#include <diopi/diopirt.h>

#include <string>
#include <mutex>
#include <string>
#include <utility>

namespace impl {
Expand Down

0 comments on commit 1f508de

Please sign in to comment.