Merge pull request #15 from samdoghor/main

fix: checks if files end with newline. closes #13

alxcheck/checks/general.py

@@ -18,5 +18,6 @@ def check_file_not_empty(file_path):
 def check_file_ends_with_new_line(file_path):
     if not check_file_not_empty(file_path):
         return False
-    with open(file_path, "r") as f:
-        return f.read()[-1] == "\n"
+
+    with open(file_path, "rb") as f:
+        return f.readlines()[-1] == b"\n"

alxcheck/main.py

@@ -11,6 +11,8 @@ def main():
             sys.exit(1)
         if not check_file_not_empty("README.md"):
             print_file_empty("README.md")
+        if not check_file_ends_with_new_line("README.md"):
+            print_no_ending_new_line("README.md")
         for root, dirs, files in os.walk("."):
             # exclude virtual environment folders
             for dir in dirs:
@@ -23,7 +25,8 @@ def main():
             for file in files:
                 file_path = os.path.join(root, file)
                 if file_path.endswith(
-                    (".c", ".py", ".js", ".m", ".h", ".mjs", ".jsx", ".json")
+                    (".c", ".py", ".js", ".m", ".h",
+                     ".mjs", ".jsx", ".json")
                 ):
                     if not check_file_ends_with_new_line(file_path):
                         if not is_empty_init_py(file_path):