-
Notifications
You must be signed in to change notification settings - Fork 991
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fuzzy_time segment prints "o'clock three" instead of "three o'clock" #2257
Comments
Hi @doctor-jan, Thanks for your report. Indeed, that looks like a bug. There was actually even another bug with the string used for "half-past": x:31 and x:30 were printed as "half past x+1". (With such bugs being unnoticed for this long, I am seriously wondering if anyone is actually using this segment.) The fix is a little bit different than what you suggested:
|
Thanks for your super-speedy fix :-) This is a much better solution, and it looks like the code is now a bit less complex. |
When using the fuzzy_time option in the time.py segment, at the top of each hour we see, for example,
o'clock three
instead of
three o'clock
All other times are in the format 'x to three' or 'x past three'. So extra code is needed to detect the 0 minutes times and swap the components around.
The text was updated successfully, but these errors were encountered: