jorisvandenbossche commented on code in PR #12930:
URL: https://github.com/apache/arrow/pull/12930#discussion_r853887418
##########
python/pyarrow/tests/strategies.py:
##########
@@ -29,6 +30,11 @@
import zoneinfo
except ImportError:
zoneinfo = None
+if sys.platform == 'win32':
+ try:
+ import tzdata
+ except ImportError:
+ tzdata = None
Review Comment:
Potential idea to simplify this: in this case we could also set `zoneinfo =
None`, and keep the below code similar (then we only need to check `zoneinfo`
to see if the test should be skipped or not). We could then just update the
skip message to something like "no module named zoneinfo (or tzdata on Windows)"
--
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]
