Restore files modified for testing purposes(new)

Author: dejavudwh

.github/workflows/file_check.yml

@@ -1,23 +1,20 @@
-name: Check File Format, License and Static Code Analysis(cppcheck)
+name: Check File Format and License
 
 on: [pull_request]
 
 jobs:
   scancode_job:
     runs-on: ubuntu-latest
-    name: Scan code format, license and static code analysis
+    name: Scan code format and license
     steps:
       - uses: actions/checkout@v3
       - name: Set up Python
         uses: actions/setup-python@v3
         with:
           python-version: 3.8
 
-      - name: Check Format, License and Static Code Analysis(cppcheck)
+      - name: Check Format and License
         shell: bash
         run: |
-          sudo apt-get update
-          sudo apt-get -qq install cppcheck
-          cppcheck --version
           pip install click chardet PyYaml
           python tools/ci/file_check.py check 'https://github.com/RT-Thread/rt-thread' 'master'

.github/workflows/static_code_analysis.yml

@@ -1,9 +1,9 @@
 name: Static code analysis
 
-on: [push]
-#   pull_request:
-#     branches:
-#       - master
+on:
+  pull_request:
+    branches:
+      - master
 
 jobs:
   scancode_job:
@@ -27,4 +27,4 @@ jobs:
           cppcheck --version
           ls
           git branch -a
-          python tools/ci/cpp_check.py check 'https://github.com/RT-Thread/rt-thread' 'master'
+          python tools/ci/cpp_check.py check

bsp/wch/risc-v/.ignore_format.yml

@@ -4,4 +4,3 @@
 
 dir_path:
 - Libraries/CH32V10x_StdPeriph_Driver
-- Libraries/ch56x_drivers

bsp/wch/risc-v/Libraries/ch56x_drivers/ch56x_gpio.c

@@ -186,7 +186,7 @@ static rt_base_t gpio_pin_get(const char *name)
             }
             if (pin < 32 && (pin_ports[port].pin_mark & (1 << pin)))
             {
-                return port * 32 + pin;	  									
+                return port * 32 + pin;
             }
         }
     }

tools/ci/cpp_check.py

@@ -5,7 +5,7 @@
 #
 # Change Logs:
 # Date           Author       Notes
-# 2021-04-01     LiuKang      the first version
+# 2023-05-16     dejavudwh    the first version
 #
 
 import click
@@ -36,25 +36,13 @@ def cli(ctx):
     pass
 
 @cli.command()
-@click.argument(
-    'repo',
-    nargs=1,
-    type=click.STRING,
-    default='https://github.com/RT-Thread/rt-thread',
-)
-@click.argument(
-    'branch',
-    nargs=1,
-    type=click.STRING,
-    default='master',
-)
-def check(repo, branch):
+def check():
     """
     static code analysis(cppcheck).
     """
     format_ignore.init_logger()
     # get modified files list
-    checkout = format_ignore.CheckOut(repo, branch)
+    checkout = format_ignore.CheckOut()
     file_list = checkout.get_new_file()
     if file_list is None:
         logging.error("checkout files fail")

tools/ci/file_check.py

@@ -1,5 +1,5 @@
 #
-# Copyright (c) 2006-2023, RT-Thread Development Team
+# Copyright (c) 2006-2022, RT-Thread Development Team
 #
 # SPDX-License-Identifier: Apache-2.0
 #
@@ -16,7 +16,6 @@
 import chardet
 import logging
 import datetime
-import subprocess
 
 def init_logger():
     log_format = "[%(filename)s %(lineno)d %(levelname)s] %(message)s "
@@ -221,21 +220,6 @@ def check(self):
 
         return check_result
 
-class CPPCheck:
-    def __init__(self, file_list):
-        self.file_list = file_list
-        
-    def check(self):
-        file_list_filtered = [file for file in self.file_list if file.endswith(('.c', '.cpp', '.cc', '.cxx'))]
-        logging.info("Start to static code analysis.")
-        check_result = True
-        for file in file_list_filtered:
-            result = subprocess.run(['cppcheck', '--enable=warning', 'performance', 'portability', '--inline-suppr', '--error-exitcode=1', '--force', file], stdout = subprocess.PIPE, stderr = subprocess.PIPE)
-            logging.info(result.stdout.decode())
-            logging.info(result.stderr.decode())
-            if result.stderr:
-                check_result = False
-        return check_result
 
 @click.group()
 @click.pass_context
@@ -266,7 +250,7 @@ def cli(ctx):
 )
 def check(check_license, repo, branch):
     """
-    check files license, format and static code analysis(cppcheck).
+    check files license and format.
     """
     init_logger()
     # get modified files list
@@ -279,15 +263,13 @@ def check(check_license, repo, branch):
     # check modified files format
     format_check = FormatCheck(file_list)
     format_check_result = format_check.check()
-    cpp_check = CPPCheck(file_list)
-    cpp_check_result = cpp_check.check()
     license_check_result = True
     if check_license:
         license_check = LicenseCheck(file_list)
         license_check_result = license_check.check()
 
-    if not format_check_result or not cpp_check_result or not license_check_result:
-        logging.error("file format check or license check or static code analysis(cppcheck) fail.")
+    if not format_check_result or not license_check_result:
+        logging.error("file format check or license check fail.")
         sys.exit(1)
     logging.info("check success.")
     sys.exit(0)

tools/ci/format_ignore.py

@@ -1,3 +1,13 @@
+#
+# Copyright (c) 2006-2023, RT-Thread Development Team
+#
+# SPDX-License-Identifier: Apache-2.0
+#
+# Change Logs:
+# Date           Author       Notes
+# 2023-05-16     dejavudwh    the first version
+#
+
 import yaml
 import logging
 import os
@@ -12,11 +22,9 @@ def init_logger():
                         )
 
 class CheckOut:
-    def __init__(self, rtt_repo, rtt_branch):
-        self.root = os.getcwd()
-        self.rtt_repo = rtt_repo
-        self.rtt_branch = rtt_branch
-
+    def __init__(self):
+        pass
+    
     def __exclude_file(self, file_path):
         dir_number = file_path.split('/')
         ignore_path = file_path