paleolimbot commented on PR #417: URL: https://github.com/apache/arrow-nanoarrow/pull/417#issuecomment-2048309273
Thank you for the detailed look! > For the zoneinfo vs dateutil, we could also bump the minimum Python version from 3.8 to 3.9 I am not sure that `zoneinfo` is available on enscripten/pyodide (a brief check suggested that dateutil via micropip works but zoneinfo does not). > For timezones, there is one aspect not covered by this PR. The Arrow spec also allows fixed offsets of the form "+XX:XX" or "-XX:XX". Good catch! This wasn't too bad to stick into the existing timezone resolver so I added it + a test case! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
