Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix cargo audit issues #1137

Merged
merged 5 commits into from
Aug 19, 2024
Merged

Conversation

qtfkwk
Copy link
Contributor

@qtfkwk qtfkwk commented Aug 17, 2024

Related to: #901

Please review individual commits...

tldr:

  • Ran cargo outdated; cargo audit
  • Updated dependencies via cargo upgrade -i && cargo update
  • Fixed issues preventing build (mostly due to clap v3 => v4)
  • Fixed issues from cargo clippy -- -D clippy::all
  • Cleaned up

@XAMPPRocky
Copy link
Owner

Thank you for your PR and congrats on your first contribution! 🎉

@XAMPPRocky XAMPPRocky merged commit 9627a24 into XAMPPRocky:master Aug 19, 2024
71 checks passed
This was referenced Aug 20, 2024
slotThe added a commit to slotThe/tokei that referenced this pull request Nov 10, 2024
The new value parser introduced in [1] did not parse the option into a
`Format`, but rather into a string, thus breaking alternative output
formats, as a different type was expected.

Fixes: XAMPPRocky#1170

[1]: 9627a24
     (Fix cargo audit issues (XAMPPRocky#1137))
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants