Skip to content
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.

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

Created Enemies by Lauv #324

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Created Enemies by Lauv #324

wants to merge 1 commit into from

Conversation

02zchen
Copy link

@02zchen 02zchen commented May 14, 2019

This is my first time contributing to this repo, I read the contribution instruction but wanted to make I was doing this correctly before proceeding.

@snshn
Copy link
Member

snshn commented May 15, 2019

Hey @02zchen thank you for the contribution!
The text looks good, but the branch is unable to pass 2 automated tests, here's the long: https://travis-ci.com/Lyrics/lyrics/jobs/200190669

The first one is the absence of album directory, currently the file is located where only directories can be located. If this song has been released outside of album, please feel free to put it into a directory named after the song, that's what contributors have been doing so far in those cases. MusicBrainz tells me the album name is I Met You When I Was 18 (The Playlist) but I may be off, not too familiar with that artist's works.

The other failing test is the lack of metadata, it's needed to make the file contain description of what song it is in case it ever gets taken out of the database. Here's a basic example of metadata: https://github.com/Lyrics/lyrics/blob/master/database/A/Alice%20In%20Chains/Dirt/Would#L35

@02zchen
Copy link
Author

02zchen commented May 15, 2019 via email

@snshn snshn added the work-in-progress Not ready label Dec 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants