Skip to content

Commit 3f6352d

Browse files
Revert "chore: use annotations instead of in house linter (#2905)"
This reverts commit 41653de.
1 parent 41653de commit 3f6352d

File tree

2 files changed

+42
-16
lines changed

2 files changed

+42
-16
lines changed

.github/workflows/approved-label.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,10 @@ jobs:
55
name: Add "approved" label when approved
66
runs-on: ubuntu-latest
77
steps:
8-
- name: Add "approved" label when approved
9-
uses: abinoda/label-when-approved-action@v1.0.7
10-
env:
11-
APPROVALS: "1"
12-
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
13-
ADD_LABEL: "approved"
14-
REMOVE_LABEL: ""
8+
- name: Add "approved" label when approved
9+
uses: pullreminders/label-when-approved-action@master
10+
env:
11+
APPROVALS: "1"
12+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
13+
ADD_LABEL: "approved"
14+
REMOVE_LABEL: ""

.github/workflows/awesome_workflow.yml

Lines changed: 35 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -9,14 +9,42 @@ jobs:
99
runs-on: ubuntu-latest
1010
steps:
1111
- uses: actions/checkout@v4
12-
- uses: cpp-linter/cpp-linter-action@v2
13-
id: linter
14-
env:
15-
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
1612
with:
17-
style: "file"
18-
tidy-checks: ".clang-tidy"
19-
thread-comments: ${{ github.event_name == 'pull_request' && 'update' }}
13+
fetch-depth: 0
14+
- uses: actions/setup-python@v4
15+
- name: requirements
16+
run: |
17+
sudo apt-get -qq update
18+
sudo apt-get -qq install clang-tidy clang-format
19+
# checks are passing with less errors when used with this version.
20+
# The default installs v6.0 which did not work out well in my tests
21+
- name: Setup Git Specs
22+
run: |
23+
git config --global user.name github-actions[bot]
24+
git config --global user.email 'github-actions[bot]@users.noreply.github.com'
25+
- name: Filename Formatter
26+
run: |
27+
wget https://raw.githubusercontent.com/TheAlgorithms/scripts/main/filename_formatter.sh
28+
chmod +x filename_formatter.sh
29+
./filename_formatter.sh . .cpp,.hpp
30+
- name: Get file changes
31+
run: |
32+
git branch
33+
git diff --diff-filter=dr --name-only origin/master > git_diff.txt
34+
echo "Files changed-- `cat git_diff.txt`"
35+
- name: Configure for static lint checks
36+
# compiling first gives clang-tidy access to all the header files and settings used to compile the programs.
37+
# This will check for macros, if any, on linux and not for Windows. But the use of portability checks should
38+
# be able to catch any errors for other platforms.
39+
run: cmake -B build -S . -DCMAKE_EXPORT_COMPILE_COMMANDS=ON
40+
- name: Lint modified files
41+
shell: bash
42+
run: python3 scripts/file_linter.py
43+
- name: Commit and push changes
44+
run: |
45+
git diff DIRECTORY.md
46+
git commit -am "clang-format and clang-tidy fixes for ${GITHUB_SHA::8}" || true
47+
git push origin HEAD:$GITHUB_REF || true
2048
2149
build:
2250
name: Compile checks
@@ -31,8 +59,6 @@ jobs:
3159
- uses: actions/checkout@v4
3260
with:
3361
submodules: true
34-
- name: GCC problem matcher
35-
uses: ammaraskar/[email protected]
3662
- run: |
3763
cmake -B ./build -S .
3864
cmake --build build --parallel 4

0 commit comments

Comments
 (0)