tao icon indicating copy to clipboard operation
tao copied to clipboard

Clippy fixes

Open matthiasbeyer opened this issue 2 years ago • 3 comments

WIP!!!

This is a WIP effort to fix all clippy fixes in the project. It is based on #48, so that PR should clearly go in first, I will then rebase this one.

It is nowhere done, there's a lot reported by clippy. I am just opening this so that you can see what I am doing :sweat_smile:

matthiasbeyer avatar Mar 10 '23 08:03 matthiasbeyer

Deploy Preview for thriving-beignet-855860 ready!

Name Link
Latest commit 09daafbdc09122bcbe64e9f3ec152bb46e90794a
Latest deploy log https://app.netlify.com/sites/thriving-beignet-855860/deploys/640b058083eb0400085a310c
Deploy Preview https://deploy-preview-49--thriving-beignet-855860.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar Mar 10 '23 08:03 netlify[bot]

Cleaned up, rebased and "clippy-clean" IMO.

#48 should go in first still, though! :laughing:

matthiasbeyer avatar Mar 10 '23 10:03 matthiasbeyer

This PR seems to remove several things that were unused, but with intent to use later. While I think clippy is usually a useful tool and very useful on projects in a more mature state, I don't think many of these changes do much to help this particular codebase, with some actually doing it some harm.

zesterer avatar Mar 10 '23 20:03 zesterer