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.

Sign up for GitHub

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

Jump to bottom

Refactoring #22

Merged

kesslern merged 3 commits into kesslern:master from jonvmey:refactoring

Oct 22, 2020

Contributor

jonvmey commented Oct 15, 2020

Just some minor refactorings.

jvandermey-evertz added 3 commits

October 14, 2020 23:00


          Avoid repeatedly building ignore list

02abf2b

Build the list once and reuse it instead of rebuilding it for every
entry in WalkDir.


          Extract iterator expression to variable

4ac0f84


          Replace unwraps with explanatory expects

a870493

kesslern approved these changes

View reviewed changes

Owner

kesslern left a comment

👍 💯

kesslern merged commit 5d5aac5 into kesslern:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet