Merge branch 'main' into arshitas/remove-unused-imports

Author: Dheerajyadav1

.gitignore

@@ -1 +1,9 @@
-node_modules
+node_modules
+# Ignore virtual environments
+myenv/
+venv/
+.env
+.venv
+ENV/
+env.bak/
+venv.bak/

app/__init__.py

@@ -0,0 +1 @@
+# i just created this file so python know this directory is package and can import from it (needed to import extract function in root/tests/test_extract.py)

app/__pycache__/__init__.cpython-310.pyc

@@ -0,0 +1 @@
+# i just created this file so python know this directory is package and can import from it (needed to import extract function in root/tests/test_extract.py)

app/etl/__init__.py

@@ -20,7 +20,7 @@ def extract(path: str = "xyz.csv") -> pd.DataFrame :
     if not os.path.exists(path):
         raise FileNotFoundError(f"❌ File not found: {path}")
 
-    if not path.lower().endswith('.csv'):  # TODO (Find & Fix)
+    if not str(path).lower().endswith('.csv'):
         raise ValueError(f"❌ File must be a CSV: {path}")
 
     try:
@@ -31,12 +31,16 @@ def extract(path: str = "xyz.csv") -> pd.DataFrame :
         for encoding in encodings:
             try:
                 # TODO (Find & Fix)
-                pass
+                pass 
             except UnicodeDecodeError:
                 print(f"Failed to read with encoding '{encoding}'")  # Log the encoding that failed
+                continue
+            except Exception as e:
+                print(f"Error reading with encoding '{encoding}': {e}")
+                continue
 
         if df is None:
-            raise ValueError(f" Could not read CSV with tried encodings: {encodings}")
+            raise ValueError(f"Could not read CSV with tried encodings: {encodings}")
 
         # Validate data
         if df.empty:

app/etl/__pycache__/__init__.cpython-310.pyc

@@ -0,0 +1 @@
+# i just created this file so python know this directory is package and can import from it (needed to import extract function in root/tests/test_extract.py)