search

Aardwolf-Social / aardwolf-social

Powering connected social communities with open software.
https://aardwolf.social
GNU Affero General Public License v3.0
485 stars 44 forks source link

TODO: Cleanup of "dead code" #338

Open BanjoFox opened 1 week ago

BanjoFox commented 1 week ago

Using VS Code and the rust-analyzer extension there are numerous "dead code", and "unused import" warnings being detected. Resolving these will not only help to implement functionality, but will also go a long way towards cleaning up the visual clutter when running tests and builds.

There may be instances, such as 'id' and 'csrf', where implementing a "TODO" function would be acceptable for the interim. On the other hand there are still numerous uses of e when it comes to error handling which should be implemented sooner than later.

jakerogers-1 commented 1 week ago

Hi! I'd love to take a shot at this.

BanjoFox commented 1 week ago

Have at!

Almost everything that I have seen so far had a purpose when originally written. If that purpose is not immediately apparent, or easy to implement then create a way for future devs to quickly identify it :)

BanjoFox commented 1 week ago

Oh, and it is not necessary to fix all of it in one go. Some of that functionality could/should be split out into other tasks.