search

mysociety / mapit.mysociety.org

A Django project for running mySociety's main instance of MapIt for the UK
http://mapit.mysociety.org
Other
6 stars 2 forks source link

Switch from xlrd to openpyxl for xlsx support #122

Open davea opened 3 years ago

davea commented 3 years ago

Recent releases of xlrd no longer support xlsx files, and recommend openpxyl instead. Would be good to migrate sooner rather than later in case we need to updatexlrd for e.g. security reasons.

dracos commented 3 years ago

Just to note we're not using it directly, it's pulled in via pyexcel (and it itself pinned xlrd to <2 in https://github.com/pyexcel/pyexcel-xls/commit/ced153d95cfb666085cb0cbb6a95923b716d1294 but I didn't want to upgrade all of that).