search

JuliaText / CorpusLoaders.jl

A variety of loaders for various NLP corpora.
Other
32 stars 13 forks source link

Bump CSV compat #49

Closed jbrea closed 2 years ago

jbrea commented 2 years ago

Would be nice to use this with the new CSV version.

oxinabox commented 2 years ago

Our use of CSV is super-minimal. That should still work find now, I checked the docs. So I am going to merge even though CI is breaking.

48

jbrea commented 2 years ago

Wow, that was quick. Thanks a lot!

ablaom commented 2 years ago

@oxinabox Could we please have a patch release?

jbrea commented 2 years ago

@ablaom it's already merged https://github.com/JuliaRegistries/General/pull/44930 (I think it's just the TagBot not running on this repo).