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

Dep updates and Clippy fixes #101

Merged
merged 1 commit into from
Mar 30, 2022

Conversation

BlackDex
Copy link
Contributor

  • Updated chrono-tz to version 0.6, this fully removes all time dependencies.
  • Update once_cell to version 1.10, no major changes done, and works fine.
  • Updated lint deny to new format
  • Added clippy allow for needless_doctest_main

This will fully remove any (dev-)dependency for the time crate and it
also gets rid of warnings generated during cargo test or cargo clippy

- Updated `chrono-tz` to version 0.6, this fully removes all time dependencies.
- Update `once_cell` to version 1.10, no major changes done, and works fine.
- Updated lint deny to new format
- Added clippy allow for needless_doctest_main

This will fully remove any (dev-)dependency for the time crate and it
also gets rid of warnings generated during `cargo test` or `cargo clippy`
Copy link
Owner

@zslayton zslayton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent, thanks!

@zslayton zslayton merged commit 26a4cb7 into zslayton:master Mar 30, 2022
@BlackDex BlackDex deleted the dep-update-and-clippy-fixes branch March 30, 2022 18:14
@BlackDex
Copy link
Contributor Author

BlackDex commented Apr 4, 2022

@zslayton Would it be possible to release a v0.10.1 or something with the updated deps?
That would be really nice to have it remove the old time crate dependency from the dep-tree.

@zslayton
Copy link
Owner

zslayton commented Apr 4, 2022

Release v0.10.1 is now available on crates.io.

@BlackDex
Copy link
Contributor Author

BlackDex commented Apr 4, 2022

🎉 thanks you!

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.

2 participants