search

benadida / helios-server

Helios server
http://heliosvoting.org
Apache License 2.0
711 stars 341 forks source link

Remove unicodecsv dependency #351

Closed pauarge closed 3 months ago

pauarge commented 2 years ago

On Python 3, all strings are unicode strings. Helios has the unicodecsv dependency for reading CSV files, but that package is only intended to be used in Python 2. After upgrading Helios to Python 3, that dependency can be dropped.

After dropping the dependency, this PR tweaks the code that was using it to use the built-in csv module.

benadida commented 3 months ago

could you squash these change into a single commit? Would love to merge them.

pauarge commented 3 months ago

@benadida done