[CHORE] Added todobot config and changed all todos to remove colons

Author: Seluj78

.github/.config.yml

@@ -0,0 +1,2 @@
+todo:
+  keyword: ['@todo', 'TODO:', 'TODO', '@todo:', '@TODO', '@TODO:']

42PyChecker.py

@@ -6,7 +6,7 @@
 import argparse
 import platform
 from PyChecker.projects import libft, ft_commandements, other
-# @todo: Add verbose output
+# @todo Add verbose output
 
 
 def print_header():
@@ -82,10 +82,10 @@ def main():
     print_header()
     if args.project == "libft":
         libft.check(root_path, args)
-    # @todo: Handle options for 42commandements: No option can be passed (like --no-norm)
+    # @todo Handle options for 42commandements: No option can be passed (like --no-norm)
     if args.project == "42commandements":
         ft_commandements.check(args)
-    # @todo: Handle options for other: No option can be passed (like --no-norm)
+    # @todo Handle options for other: No option can be passed (like --no-norm)
     if args.project == "other":
         other.check(root_path, args)
 

PyChecker/projects/libft.py

@@ -88,7 +88,7 @@ def check(root_path: str, args):
     if not args.no_maintest:
         maintest_ok, maintest_fail = maintest.run_libft(args.path, root_path)
     if not args.no_libft_unit_test:
-        # @todo: Add return message/values to results for libft-unit-test
+        # @todo Add return message/values to results for libft-unit-test
         libft_unit_test.run(root_path, args)
     print("\n\n\nThe results are in:\n")
     if not args.no_author:

PyChecker/projects/other.py

@@ -10,6 +10,6 @@ def check(root_path: str, args):
         author.check(args.path)
     if not args.no_norm:
         norme.check(args.path, root_path)
-    # @todo: Add the makefile check and find the binary name itself
-    # @todo: Add forbidden function check based on assumption of what the project is.
+    # @todo Add the makefile check and find the binary name itself
+    # @todo Add forbidden function check based on assumption of what the project is.
     return 0

PyChecker/testing_suite/libft_unit_test.py

@@ -55,4 +55,4 @@ def run(root_path: str, args):
             result = subprocess.run([root_path + '/testing_suite/libft-unit-test/run_test', '-b'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT).stdout.decode('utf-8')
             print(result)
             file.write(result)
-    # @todo: return results from benchmark and tests
+    # @todo return results from benchmark and tests

PyChecker/testing_suite/maintest.py

@@ -44,7 +44,7 @@ def run_libft(project_path: str, root_path: str):
         # @todo Add a check to handle libft where file aren't at libft/ but can be in libft/src
         if not os.path.exists(project_path + '/ft_' + file + '.c'):
             missing_functions.append(file)
-    # @todo: special case for memalloc and memdel
+    # @todo special case for memalloc and memdel
     comment_define(root_path + '/testing_suites/Maintest/libft/main.c', root_path + '/libft_main.c', missing_functions)
     with open(root_path + "/.mymaintest", 'w+') as file:
         file.write("*------------------------------------------------------*\n")
@@ -62,7 +62,7 @@ def run_libft(project_path: str, root_path: str):
         print(result)
         result = subprocess.run([root_path + '/libft_main.out'], stdout=subprocess.PIPE,
                                 stderr=subprocess.STDOUT).stdout.decode('utf-8')
-        # @todo: Count number of OK and FAILs and yellow tests to get score for maintest
+        # @todo Count number of OK and FAILs and yellow tests to get score for maintest
         file.write(result + '\n')
         print(result)
     if platform.system() == "Linux":

PyChecker/utils/makefile.py

@@ -78,7 +78,7 @@ def check_makefile_all(project_path: str, binary_name: str, root_path: str):
                 print("-> Error when processing rule `all':"
                            " It should NOT have removed *.o")
                 error_count += 1
-            # @todo  [ -z "$(echo ${MAKEALLTWICE} | grep -i "Nothing to be done")" -a -z "$(echo ${MAKEALLTWICE} | grep -i "is up to date")" ] && printf "%s\n" "-> Failing rule: Processing the rule 'all' twice in a row should result in nothing to be done" && RET=1
+            # @todo [ -z "$(echo ${MAKEALLTWICE} | grep -i "Nothing to be done")" -a -z "$(echo ${MAKEALLTWICE} | grep -i "is up to date")" ] && printf "%s\n" "-> Failing rule: Processing the rule 'all' twice in a row should result in nothing to be done" && RET=1
     return error_count
 
 
@@ -253,7 +253,7 @@ def check_makefile_name(project_path: str, binary_name: str, root_path: str):
                 print("-> Error when processing rule `fclean': It should"
                            " NOT have removed *.o\n")
                 error_count += 1
-            # @todo  [ -z "$(echo ${MAKEALLTWICE} | grep -i "Nothing to be done")" -a -z "$(echo ${MAKEALLTWICE} | grep -i "is up to date")" ] && printf "%s\n" "-> Failing rule: Processing the rule 'all' twice in a row should result in nothing to be done" && RET=1
+            # @todo [ -z "$(echo ${MAKEALLTWICE} | grep -i "Nothing to be done")" -a -z "$(echo ${MAKEALLTWICE} | grep -i "is up to date")" ] && printf "%s\n" "-> Failing rule: Processing the rule 'all' twice in a row should result in nothing to be done" && RET=1
     return error_count
 
 

PyChecker/utils/static.py

@@ -13,6 +13,6 @@ def check(root_path: str, args):
         result = subprocess.run(['sh', root_path + '/scripts/check_static.sh', args.path],
                                 stdout=subprocess.PIPE).stdout.decode('utf-8')
         file.write(result)
-    # @todo: Replace check_static script with a python way of doing it
+    # @todo Replace check_static script with a python way of doing it
     print(result)
     return result