Skip to content

Update dependencies and fixing some clippy warnings#1

Open
enaut wants to merge 6 commits intoagrover:mainfrom
enaut:dependency-clippy
Open

Update dependencies and fixing some clippy warnings#1
enaut wants to merge 6 commits intoagrover:mainfrom
enaut:dependency-clippy

Conversation

@enaut
Copy link
Copy Markdown

@enaut enaut commented Dec 9, 2023

Updated the dependency to nom 7

Fixed some clippy warnings mostly:

  • redundant code (closures that only call one function can be the function instead)
  • complex types, solved by introducing type aliases

@enaut
Copy link
Copy Markdown
Author

enaut commented Dec 12, 2023

Ouch now I pushed some extra commits to that branch... Take a close look at the last two commits... If you really want thiserror as a dependency I can redo the PR without if you want to.

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.

1 participant