Skip to content
This repository was archived by the owner on Feb 20, 2024. It is now read-only.

refactor: cleanup the api_app lib from clippy warnings#23

Merged
lsipii merged 2 commits into
mainfrom
refactor/fix-clippy-warnings
Feb 9, 2023
Merged

refactor: cleanup the api_app lib from clippy warnings#23
lsipii merged 2 commits into
mainfrom
refactor/fix-clippy-warnings

Conversation

@lsipii
Copy link
Copy Markdown
Contributor

@lsipii lsipii commented Feb 8, 2023

  • cleaned the api_app with suggestions from cargo clippy
    • warnings visible when clippy ran against the library folder instead of project root

@lsipii lsipii requested a review from lassipatanen February 8, 2023 17:42
@lsipii
Copy link
Copy Markdown
Contributor Author

lsipii commented Feb 8, 2023

Ajoin rustin clippy:n tuota pääasiaa eli api_app-kirjastoa vasten ja fiksasin syntaksit sen varoitusten ja korjausehdotusten mukaan. Jos cargo clippyä ajeskelee projektin juuressa niin se ei suoraan lähde ehdottelemaan parannuksia alla oleviin kirjastototeutuksiin, jossa tämän projektin pääasiat sijaitsevat. Sain varoitukset aikaiseksi navigoimalla src/lib/api_app/ ja ajamalla siellä cargo clippy. Harmi että myöskään käyttämäni vscode:n rust-analyzer ei vinkkaile yhtä tehokkaasti kuin tuo clippy tekee. Linkkasin @lassipatanen tsekkailemaan että mitä syntaksiehdotuksia clippy aikaisempaan totetutukseen tarjosi.

Pääasiassa warningit olivat turhia returneja ja sellaista, mutta myös turhia tiedon kopiointeja ja tyypityksessä päsmäröintejä.

Copy link
Copy Markdown
Contributor

@lassipatanen lassipatanen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hyvvä duunia! 👍 Clippy toki osaa Rustia paremmin kuin itse, eli uskotaan noiden muutoksien olevan hyviä. :D

@lsipii lsipii merged commit 925ee7b into main Feb 9, 2023
@lsipii lsipii deleted the refactor/fix-clippy-warnings branch February 9, 2023 17:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants