Refactor: Improve clarity and readability of loader function

[Nate1729]

Description here

There are some readability improvements that could be made to the loader() function in core/loader:loader. For example checking if some_string == '' is redundant. An empty string is cast to False in python (L27).

On L29 rather than use a break a continue would be more appropriate since you want to go to the next iteration rather than stop the iteration entirely.

L37 serves no purpose to be inside the main for-loop. It could be defined even before the with-block.

L39 could be replaced with a call to the filter() function which removes the need to copy the data into a new list.

branch suggestion: refactor/loader:loader
commit suggestion: refactor: #64 - refactor loader fn
we like to see conventional commits

[RackReaver]

Is this a bug or just a code improvement?

[Nate1729]

Is this a bug or just a code improvement?

This is just a code improvement