Fix in case a custom protocol was registered in fsspec

petl-developers / petl

Python Extract Transform and Load Tables of Data

MIT License

1.24k stars 193 forks source link

Fix in case a custom protocol was registered in fsspec #647

Closed timheb closed 1 year ago

timheb commented 1 year ago

This PR fix the issue described in #646. In case a custom fsspec protocol is registered the "spec" is not of type dict. Implemented the suggested solution in #646 to check first if "spec" is of type dict.

mlemainque commented 1 year ago

LGTM 👍🏻

juarezr commented 1 year ago

Merging as the change is trivial.
Failures are due to:
- python2.7 not included in ubuntu-latest anymore
- new bugs with SQLAlchemy and MySQL