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

Feature/minor refactor #46

Open

berkes wants to merge 2 commits into Publidata:master from placebazaar:feature/minor-refactor

Contributor

berkes commented Dec 18, 2019

Using Ruby-isms to make the code easier to understand.

(All while I'm trying to understand it)

berkes added 2 commits

December 18, 2019 21:07


          Use guard clause instead of huge if-else

250f180


          Use Ruby-isms to compact and filter loops.

b1d1aef

Instead of next- or if/else in the loop block, prefilter the items
to be looped through.

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

Labels

None yet