mirror of
https://github.com/TheAlgorithms/C
synced 2024-11-21 21:11:57 +03:00
chore: add the linter to a separate Python script (#1272)
* updating DIRECTORY.md * chore: add the linter to a separate Python script --------- Co-authored-by: github-actions[bot] <github-actions@users.noreply.github.com>
This commit is contained in:
parent
1bbbac6546
commit
518a7cafd5
44
.github/workflows/awesome_workflow.yml
vendored
44
.github/workflows/awesome_workflow.yml
vendored
@ -42,48 +42,8 @@ jobs:
|
|||||||
# be able to catch any errors for other platforms.
|
# be able to catch any errors for other platforms.
|
||||||
run: cmake -B build -S . -DCMAKE_EXPORT_COMPILE_COMMANDS=ON
|
run: cmake -B build -S . -DCMAKE_EXPORT_COMPILE_COMMANDS=ON
|
||||||
- name: Lint modified files
|
- name: Lint modified files
|
||||||
shell: python
|
shell: bash
|
||||||
run: |
|
run: python3 scripts/file_linter.py
|
||||||
import os
|
|
||||||
import subprocess
|
|
||||||
import sys
|
|
||||||
|
|
||||||
print("Python {}.{}.{}".format(*sys.version_info)) # Python 3.8
|
|
||||||
with open("git_diff.txt") as in_file:
|
|
||||||
modified_files = sorted(in_file.read().splitlines())
|
|
||||||
print("{} files were modified.".format(len(modified_files)))
|
|
||||||
|
|
||||||
cpp_exts = tuple(".c .c++ .cc .cpp .cu .cuh .cxx .h .h++ .hh .hpp .hxx".split())
|
|
||||||
cpp_files = [file for file in modified_files if file.lower().endswith(cpp_exts)]
|
|
||||||
print(f"{len(cpp_files)} C++ files were modified.")
|
|
||||||
if not cpp_files:
|
|
||||||
sys.exit(0)
|
|
||||||
subprocess.run(["clang-tidy", "-p=build", "--fix", *cpp_files, "--"],
|
|
||||||
check=True, text=True, stderr=subprocess.STDOUT)
|
|
||||||
subprocess.run(["clang-format", "-i", *cpp_files],
|
|
||||||
check=True, text=True, stderr=subprocess.STDOUT)
|
|
||||||
|
|
||||||
upper_files = [file for file in cpp_files if file != file.lower()]
|
|
||||||
if upper_files:
|
|
||||||
print(f"{len(upper_files)} files contain uppercase characters:")
|
|
||||||
print("\n".join(upper_files) + "\n")
|
|
||||||
|
|
||||||
space_files = [file for file in cpp_files if " " in file or "-" in file]
|
|
||||||
if space_files:
|
|
||||||
print(f"{len(space_files)} files contain space or dash characters:")
|
|
||||||
print("\n".join(space_files) + "\n")
|
|
||||||
|
|
||||||
nodir_files = [file for file in cpp_files if file.count(os.sep) != 1 and "project_euler" not in file and "data_structure" not in file]
|
|
||||||
if len(nodir_files) > 1:
|
|
||||||
nodir_file_bad_files = len(nodir_files) - 1
|
|
||||||
print(f"{len(nodir_files)} files are not in one and only one directory:")
|
|
||||||
print("\n".join(nodir_files) + "\n")
|
|
||||||
else:
|
|
||||||
nodir_file_bad_files = 0
|
|
||||||
|
|
||||||
bad_files = nodir_file_bad_files + len(upper_files + space_files)
|
|
||||||
if bad_files:
|
|
||||||
sys.exit(bad_files)
|
|
||||||
- name: Commit and push changes
|
- name: Commit and push changes
|
||||||
run: |
|
run: |
|
||||||
git diff DIRECTORY.md
|
git diff DIRECTORY.md
|
||||||
|
1
.gitignore
vendored
1
.gitignore
vendored
@ -3,3 +3,4 @@
|
|||||||
*.out
|
*.out
|
||||||
.vscode/
|
.vscode/
|
||||||
build/
|
build/
|
||||||
|
git_diff.txt
|
||||||
|
40
scripts/file_linter.py
Normal file
40
scripts/file_linter.py
Normal file
@ -0,0 +1,40 @@
|
|||||||
|
import os
|
||||||
|
import subprocess
|
||||||
|
import sys
|
||||||
|
|
||||||
|
print("Python {}.{}.{}".format(*sys.version_info)) # Python 3.8
|
||||||
|
with open("git_diff.txt") as in_file:
|
||||||
|
modified_files = sorted(in_file.read().splitlines())
|
||||||
|
print("{} files were modified.".format(len(modified_files)))
|
||||||
|
|
||||||
|
cpp_exts = tuple(".c .c++ .cc .cpp .cu .cuh .cxx .h .h++ .hh .hpp .hxx".split())
|
||||||
|
cpp_files = [file for file in modified_files if file.lower().endswith(cpp_exts)]
|
||||||
|
print(f"{len(cpp_files)} C++ files were modified.")
|
||||||
|
if not cpp_files:
|
||||||
|
sys.exit(0)
|
||||||
|
subprocess.run(["clang-tidy", "-p=build", "--fix", *cpp_files, "--"],
|
||||||
|
check=True, text=True, stderr=subprocess.STDOUT)
|
||||||
|
subprocess.run(["clang-format", "-i", *cpp_files],
|
||||||
|
check=True, text=True, stderr=subprocess.STDOUT)
|
||||||
|
|
||||||
|
upper_files = [file for file in cpp_files if file != file.lower()]
|
||||||
|
if upper_files:
|
||||||
|
print(f"{len(upper_files)} files contain uppercase characters:")
|
||||||
|
print("\n".join(upper_files) + "\n")
|
||||||
|
|
||||||
|
space_files = [file for file in cpp_files if " " in file or "-" in file]
|
||||||
|
if space_files:
|
||||||
|
print(f"{len(space_files)} files contain space or dash characters:")
|
||||||
|
print("\n".join(space_files) + "\n")
|
||||||
|
|
||||||
|
nodir_files = [file for file in cpp_files if file.count(os.sep) != 1 and "project_euler" not in file and "data_structure" not in file]
|
||||||
|
if len(nodir_files) > 1:
|
||||||
|
nodir_file_bad_files = len(nodir_files) - 1
|
||||||
|
print(f"{len(nodir_files)} files are not in one and only one directory:")
|
||||||
|
print("\n".join(nodir_files) + "\n")
|
||||||
|
else:
|
||||||
|
nodir_file_bad_files = 0
|
||||||
|
bad_files = nodir_file_bad_files + len(upper_files + space_files)
|
||||||
|
|
||||||
|
if bad_files:
|
||||||
|
sys.exit(bad_files)
|
Loading…
Reference in New Issue
Block a user