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

Clarify format option for disc template #353

Merged
merged 1 commit into from
Jan 17, 2019

Conversation

duskmoss
Copy link
Contributor

The previous text implied to me that this was the sort name for the Album/Disc title. Not the artist sort name.

The previous text implied to me that this was the sort name for the Album/Disc title.  Not the artist sort name.
@welcome
Copy link

welcome bot commented Jan 14, 2019

💖 Thanks for opening your first pull request here! 💖

@JoeLametta JoeLametta merged commit 752b485 into whipper-team:master Jan 17, 2019
@welcome
Copy link

welcome bot commented Jan 17, 2019

Congrats on merging your first pull request! 🎉🎉🎉

@JoeLametta
Copy link
Collaborator

Sorry, I was tired and noticed that this pull request was based on master only after the merge. Won't revert it but next time please use the develop branch as base for all the pull requests.

Thanks,
JoeLametta

@duskmoss
Copy link
Contributor Author

duskmoss commented Jan 17, 2019 via email

@JoeLametta
Copy link
Collaborator

Maybe my first pull request on develop can be adding some branching information to the readme ;)

👍 Yeah, maybe you can expand what's written in this section.

Cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants