search

MohamedYasser97 / lyricurator

A bot that scrapes azlyrics.com for lyrics and other data with additional tweeting functionality
https://twitter.com/lyricurator
The Unlicense
5 stars 8 forks source link

Refactoring lyrics_util.py #32

Open jbarbaran opened 1 year ago

jbarbaran commented 1 year ago

This is my first proposal, my main objective has been to separate functionality by concerns and also decrease the number of lines of lyrics_util.py, and at the same time removing code repeated.

Hope you find this interesting!

jbarbaran commented 1 year ago

By the way... I forgot to mention before, this PR addresses issue 10!

jbarbaran commented 1 year ago

Thank you @MohamedYasser97 I considered using object abstraction but I don't think it is worth it with the current scope of the project.

I'm glad I have the opportunity to contribute to the project!