Merge pull request #29 from Eligoze75/deployment_file

Deployment file

Author: Eligoze75

.github/workflows/deploy.yml

@@ -0,0 +1,55 @@
+name: publish-test-pypi
+
+on:
+  push:
+    branches: [ master ]
+  workflow_dispatch:
+
+jobs:
+  unit-test-pytest:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Check-out repository
+        uses: actions/checkout@v2
+
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: "3.12"
+
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install -e .[tests]
+
+      - name: Run tests with coverage
+        run: |
+          pytest --cov --cov-report=term --cov-branch
+
+  pypi-test:
+    needs: unit-test-pytest
+    runs-on: ubuntu-latest
+    
+    steps:
+      - name: Check-out repository
+        uses: actions/checkout@v2
+
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: "3.12"
+
+      - name: Install hatch
+        run: |
+          python -m pip install --upgrade pip
+          pip install -e .[dev]
+
+      - name: Build package
+        run: hatch build
+
+      - name: Publish to Test PyPI
+        uses: pypa/gh-action-pypi-publish@release/v1
+        with:
+          repository-url: https://test.pypi.org/legacy/
+          password: ${{ secrets.TEST_PYPI_API_TOKEN }}

.github/workflows/release.yml

@@ -4,7 +4,7 @@ on:
   pull_request:
   push:
     branches:
-      - main
+      - master
   release:
     types: [published]
 

pyproject.toml

@@ -23,7 +23,7 @@ authors = [
     { name = "Eli Gonzalez, Gurleen Kaur, Gloria Yi, Mantram Sharma", email = "[email protected]" },
 ]
 license = "MIT"
-requires-python = ">= 3.10" # Adjust based on the minimum version of Python that you support
+requires-python = ">= 3.11" # Adjust based on the minimum version of Python that you support
 readme = {"file" = "README.md", "content-type" = "text/markdown"}
 # Please consult https://pypi.org/classifiers/ for a full list.
 classifiers = [

src/autoeda/get_summary_df.py

@@ -41,7 +41,7 @@ def get_summary_df(df, categorical_columns, numerical_columns):
         raise ValueError("No columns provided to attain a summary")
 
     for column in categorical_columns:
-        if df.loc[:, column].dtype != "object":
+        if df.loc[:, column].dtype not in ["object", "string"]:
             raise ValueError(f"This is not a categorical column: {column}")
         else:
             categorical_df[column] = pd.DataFrame(

tests/test_get_summary_df.py tests/unit/test_get_summary_df.pytests/test_get_summary_df.py renamed to tests/unit/test_get_summary_df.py

@@ -6,12 +6,10 @@
 errors for the incorrectly defined arguments.
 """
 
+from autoeda.get_summary_df import get_summary_df
 import pandas as pd
-import numpy as np
 import pytest
 
-from src.autoeda.get_summary_df import get_summary_df
-
 df = pd.DataFrame({
         'name': ['John', 'Jeffery', 'Alice', 'Sam', 'Millie'],
         'age': [25, 30, 18, 27, 45],