-
Notifications
You must be signed in to change notification settings - Fork 546
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
use stub for anything not unix and not windows #593
Conversation
85dbc65
to
91d1eba
Compare
Would you be able to rebase this on main? This looks like a good fix! |
8729752
to
f687bab
Compare
f687bab
to
9ac4e7b
Compare
Done |
Thanks for the quick reply, and sorry for the long wait! |
The lint errors do not seem to be caused by my changes. |
That's right, they're clippy issues from the 1.60 upgrade. Unfortunately I don't have permission to merge in spite of failing CI. Can you fix them? Otherwise I'll get to it soon. |
I have pushed a commit that silences those warnings. If you would like a different fix you can force-push to my branch or tell me what you prefer. |
1f2c563
to
51aa03e
Compare
Hopefully fixed lint for real... |
Great, thanks! |
Thank you! Much appreciated! |
Thanks for contributing to chrono!
Alternative to #568
Fixes #519
about adding the PR number)
we can't reintroduce it?