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

cargo clippy, DO NOT MERGE #16

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

cargo clippy, DO NOT MERGE #16

wants to merge 1 commit into from

Conversation

is8ac
Copy link

@is8ac is8ac commented Jan 31, 2022

I ran cargo clippy and fairly blindly did what it told me to do, and also did some other refactors that I thought looked good. cargo test passes, but I have not ran other tests. Look over the changes and pick the ones you like.

@TrevorJTClarke
Copy link
Member

@is8ac great feedback! Only a couple dumb things i forgot need a diff response that you caught and changed. Ill migrate my response to work with your changes. thanks!

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