Skip to content
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

Merged
merged 4 commits into from Apr 9, 2022

Conversation

yu-re-ka
Copy link
Contributor

@yu-re-ka yu-re-ka commented Sep 17, 2021

Thanks for contributing to chrono!

Alternative to #568
Fixes #519

  • Have you added yourself and the change to the changelog? (Don't worry
    about adding the PR number)
  • If this pull request fixes a bug, does it add a test that verifies that
    we can't reintroduce it?

@djc
Copy link
Contributor

djc commented Apr 9, 2022

Would you be able to rebase this on main? This looks like a good fix!

@yu-re-ka
Copy link
Contributor Author

yu-re-ka commented Apr 9, 2022

Would you be able to rebase this on main?

Done

@djc
Copy link
Contributor

djc commented Apr 9, 2022

Thanks for the quick reply, and sorry for the long wait!

@yu-re-ka
Copy link
Contributor Author

yu-re-ka commented Apr 9, 2022

The lint errors do not seem to be caused by my changes.

@djc
Copy link
Contributor

djc commented Apr 9, 2022

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.

@yu-re-ka
Copy link
Contributor Author

yu-re-ka commented Apr 9, 2022

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.

@yu-re-ka
Copy link
Contributor Author

yu-re-ka commented Apr 9, 2022

Hopefully fixed lint for real...

@djc djc merged commit 4983812 into chronotope:main Apr 9, 2022
@djc
Copy link
Contributor

djc commented Apr 9, 2022

Great, thanks!

@hoodmane
Copy link

hoodmane commented Apr 9, 2022

Thank you! Much appreciated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build fails on target wasm32-unknown-emscripten with: target wasm32-unknown-emscripten
3 participants