Skip to content

Commit

Permalink
fix: rmv analytcis from the tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Fabiana Clemente authored and fabclmnt committed Jul 9, 2024
1 parent 488341c commit a4afa34
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
7 changes: 6 additions & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
name: Multi-software test


env:
YDATA_PROFILING_NO_ANALYTICS: False

on:
pull_request:
push:
Expand All @@ -20,6 +22,7 @@ jobs:
pandas: ["pandas>1.1"]
numpy: ["numpy"]
matplotlib: ["matplotlib<3.9"]
analytics: False
include:
- os: ubuntu-22.04
python-version: 3.7.16
Expand Down Expand Up @@ -130,6 +133,7 @@ jobs:
pip install --upgrade pip setuptools wheel
pip install -r requirements.txt "${{ matrix.pandas }}" "${{ matrix.numpy }}"
pip install -r requirements-test.txt
echo $YDATA_PROFILING_NO_ANALYTICS
- run: make install

- run: make test_cov
Expand Down Expand Up @@ -170,6 +174,7 @@ jobs:
HADOOP_VERSION: ${{ matrix.hadoop }}
SPARK_DIRECTORY: ${{ github.workspace }}/../
SPARK_HOME: ${{ github.workspace }}/../spark/
YDATA_PROFILING_NO_ANALYTICS: ${{ matrix.analytics }}
steps:
- uses: actions/checkout@v4
- name: Setup python
Expand Down
2 changes: 1 addition & 1 deletion src/ydata_profiling/utils/logger.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ def info_def_report(self, dataframe, timeseries: bool) -> None: # noqa: ANN001
if isinstance(dataframe, pd.DataFrame):
dataframe = "pandas"
report_type = "regular"
elif isinstance(dataframe, None):
elif dataframe is None:
dataframe = "pandas"
report_type = "compare"
else:
Expand Down

0 comments on commit a4afa34

Please sign in to comment.