From cac95058297004870981c7e88948079f39e53696 Mon Sep 17 00:00:00 2001
From: Krishna Vedala <7001608+kvedala@users.noreply.github.com>
Date: Fri, 29 May 2020 23:48:29 -0400
Subject: [PATCH] commented whole repo cpplint - added modified files lint
 check

---
 .github/workflows/awesome_forkflow.yml | 77 +++++++++++++++++++++++---
 1 file changed, 69 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/awesome_forkflow.yml b/.github/workflows/awesome_forkflow.yml
index 121309d1e..07bfb99dc 100644
--- a/.github/workflows/awesome_forkflow.yml
+++ b/.github/workflows/awesome_forkflow.yml
@@ -122,21 +122,82 @@ jobs:
           git commit -am "updating DIRECTORY.md" ||  true
           git push --force origin HEAD:$GITHUB_REF || true
 
-  cpplint:
-    name: CPPLINT
-    needs: code_format
+  # cpplint:
+  #   name: CPPLINT
+  #   needs: code_format
+  #   runs-on: ubuntu-latest
+  #   steps:
+  #     - uses: actions/checkout@master
+  #     - uses: actions/setup-python@master
+  #     - run: pip install cpplint
+  #     - run: git pull
+  #     - run: cpplint --filter=-legal --recursive .
+  
+  cpplint_modified_files:
     runs-on: ubuntu-latest
+    needs: code_format
+    name: CPPLINT
     steps:
-      - uses: actions/checkout@master
-      - uses: actions/setup-python@master
-      - run: pip install cpplint
+      - uses: actions/checkout@v1 # v2 is broken for git diff
+      - uses: actions/setup-python@v1
+      - run: python -m pip install cpplint
+      - run: git remote -v
+      - run: git branch
+      - run: git remote set-url origin https://x-access-token:${{ secrets.GITHUB_TOKEN }}@github.com/$GITHUB_REPOSITORY
       - run: git pull
-      - run: cpplint --filter=-legal --recursive .
+      - run: git diff --diff-filter=dr --name-only origin/master > git_diff.txt
+      - name: cpplint_modified_files
+        shell: python
+        run: |
+          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)
+
+          print("cpplint:")
+          for cpp_file in cpp_files:
+            subprocess.run(["cpplint", "--filter=-legal/copyright", cpp_file], check=True, text=True)
+
+          print("g++:")
+          # compile_exts = tuple(".c .c++ .cc .cpp .cu .cxx".split())
+          # compile_files = [file for file in cpp_files if file.lower().endswith(compile_exts)]
+          for cpp_file in cpp_files:
+            subprocess.run(["g++", cpp_file], check=True, text=True)
+
+          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]
+          if nodir_files:
+            print(f"{len(nodir_files)} files are not in one and only one directory:")
+            print("\n".join(nodir_files) + "\n")
+
+          bad_files = len(upper_files + space_files + nodir_files)
+          if bad_files:
+            sys.exit(bad_files)
+
 
   build:
     name: Compile checks
     runs-on: ${{ matrix.os }}
-    needs: [cpplint, update_directory_md]
+    needs: [cpplint, update_directory_md, cpplint_modified_files]
     strategy:
       matrix:
         os: [ubuntu-latest, windows-latest, macOS-latest]