-
Notifications
You must be signed in to change notification settings - Fork 10
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
Example day-of-week string indexing is off-by-one in many repos #26
Comments
For now I think we can leave this one open until we take care of the rest of the instances in this list, or create separate issues on those repos |
I think it may be worth creating new ones just so they live with the repos, though the Learn Guides would live together anyway, I can handle creating them (and tackling them likely) |
@tekktrik okay, sounds good. I'm working on the change for the PCF ones now. |
Yeah, figured I'd leave it to yall to decide how best to distribute the issues among repos. Thanks for taking it over |
Looks like we're all good now! Thanks y'all! |
In examples/ds1307_simpletest.py, the list of strings used to expand the integer
time.struct_time.tm_wday
value appears to defined using the C convention (Sunday=0), rather than the Python convention (Monday=0); see https://docs.python.org/3/library/time.html#time.struct_timeA quick search reveals this error in several other repos:
The text was updated successfully, but these errors were encountered: