Redundant install instructions in CI?

[tylerriccio33]

In the github actions config we install the deps and then separately install the test deps. I feel like the first should cover the second right? I don't see why not. However, when I didn't add pytest-randomly and pytest-xdist to the test dependencies in the second step, I got errors saying they were missing, despite them being dev deps.

Any ideas?

- name: Install dependencies run: | pip install -e '.[dev]' - name: Install test dependencies run: | pip install pytest pytest-randomly pytest-xdist pytest-cov pytest-snapshot pandas polars ibis-framework[duckdb,mysql,postgres,sqlite]>=9.5.0 chatlas requests shiny

[rich-iannone]

I think we need to rework the dependencies setup in pyproject.toml. It should probably be closer in configuration to one used in Great Tables: https://github.com/posit-dev/great-tables/blob/main/pyproject.toml. And there’s some unfortunate duplication and mismatched min version requirements in our current setup (should be addressed if adopting a similar setup as GT).

I don’t think we need to keep dependency groups unless we move to UV in CI (which should be done soon).

Do you feel comfortable taking this one on?

[tylerriccio33]

Yea I can take this if you want to assign it, shouldn't be that bad

[rich-iannone]

Fixed by #94