mohamedyasser97 / lyricurator Goto Github PK

It should work like this python lyricurator.py --interval 10. This should print/tweet lyrics every 10 minutes.

Adding the CLI argument -ls or --listsongs should print all song names for the randomly selected artist instead of printing/tweeting lyrics.
If the user sets a predefined artist with -a/--artist then print all song names for that specific artist

The file lyricurator.py calls different functions based on the user's choice of CLI arguments. However, running the bot with just -hs or -hal for example won't work as in the code it requires notwitter to be set to true while it should work alone just fine

The current implementation of -la/--listartists uses the function get_artist_names which uses get_artists_links inside of it.

The problem is that get_artists_links selects a random first character of an artist's name and fetches this page for example https://www.azlyrics.com/m.html. This means that it only gets artists with names starting with m which is not "all" artists.

One solution is just loop for all letters but this is time consuming and will risk an IP ban from azlyrics.

So the required solution is to edit the argument -la/--listartists and make it accept a letter as an input and then only get artist names that start with that input character.

Note: get_artists_links is used by other functions too so be careful what you change.

Create the CLI argument -hs/--hotsongs that gets the list of hot songs on the homepage of azlyrics.com and then exits the program.

On running python lyricurator.py without any CLI arguments, it sends a tweet by default.

Instead of this behavior, it is required to force users to at least add -nt/--notwitter or the new CLI argument you would create -t/--twitter that sends the lyrics in a tweet.

If a user executes lyricurator without any arguments, print out argparse's help screen and exit the program.

Currently, it's mandatory to use either one of -nt or -t. It makes no sense to use one of those when we only want to print something in the terminal and exit like -la, -ls or the upcoming -hal and -hs.

It's required to make -nt or -t only mandatory when used with any arguments except the ones mentioned above.

Make a cli arg to get all currently available artist names.

• Add a readme file to explain how the program works with its possible argument combinations.
• Try to make it look good with different markdown styles.
• Also make a section that explains how people should contribute.
• Screenshot examples would be a nice bonus.

Create the CLI argument -hal/--hotalbums that gets the list of hot albums on the homepage of azlyrics.com and then exits the program.

It would be nice if someone could refactor this project into modules so it can just act as a CLI lyrics tool with the Twitter feature just as an added feature

Posting to twitter should be optional via a cli argument

Instead of scraping 2 or 3 lines of lyrics, the CLI argument -fs or --fullsong should get the whole song lyrics.

Notes:

• When this CLI argument is chosen, you must set the -nt/--notwitter to true.
• If the user didn't write -nt with -fs, then execute -fs and print this below/above the lyrics: 'Full song lyrics can't be tweeted'.
• If the user wrote both -nt and -fs, then just print the full song lyrics.

Current visible auth credentials are already revoked.
Auth credentials should be in a separate file that is added to .gitignore

Add a cli argument that makes the bot only fetch lyrics from the given artist