[MNT] test and clarify expected OS and python version compatibility

[fkiraly]

I would recommend to test which OS and python versions this repo is compatible with.

This can be done by adding a test matrix to the CI.

If OS and versions are not currently compatible, this should be clarified in the documentation of the repository - it may be fine to require a specific OS since this repo is app specific.

[Godkunn]

Status Update: Windows 11 Compatibility

Hi @fkiraly, I have finished testing the current develop branch on Windows 11 (Python 3.13).

I read the discussion in PR #685. While that PR fixes the initial encoding crash, it is not enough to pass the tests on Windows. The suite fails immediately after that due to environment-specific issues.

I successfully patched the environment locally and verified a clean run:
(Verified tests/connectors/zenodo/test_get_datasets_zenodo.py passes with 6 passed)

The 4 Fixes Required for Windows:

1. File Encoding: (Covered by PR [BUG] explicit encoding for file open to avoid failures on windows #685) Fixes the initial UnicodeDecodeError.

2. Dependency Failure: mysqlclient fails to build on Windows (requires MSVC).
   Fix: Hot-patched pymysql to replace MySQLdb in default_sqlalchemy.py to bypass compilation.

3. File Locking: tempfile.NamedTemporaryFile causes a PermissionError on Windows because SQLite cannot open a file Python is still holding open.
   Fix: Added a .close() call before passing the file path to the engine.

4. Missing SQLite Function: Tests generate CHECK (BINARY(name) = ...) constraints, but SQLite lacks BINARY.
   Fix: Registered a dummy BINARY function in the SQLite connection hook.

Recommendation:
We should merge #685 for the immediate fix. I can then open a follow-up PR "[MNT] Enable Windows Test Support" implementing fixes 2, 3, and 4 to get the CI fully green on Windows.

[fkiraly]

@Godkunn, great, thanks for investigating!

I think we should add a CI job for windows then, to ensure passing tests and avoiding regression for windows compatibility.

[Godkunn]

I’ve opened PR #690 to extend the CI matrix to include windows-latest.
The workflow is currently awaiting maintainer approval to run.

[Godkunn]

Hi @fkiraly — PR #690 has been updated and now includes the pytest 8 compatibility and Windows test fixes.

Since it’s from a fork, the workflow is awaiting maintainer approval to run.
Could you please approve the workflow so CI can execute?

Thanks!