-
-
Notifications
You must be signed in to change notification settings - Fork 191
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
feat: (Snippets) Remove the artists and credits sections from the sidebar #899
feat: (Snippets) Remove the artists and credits sections from the sidebar #899
Conversation
…ections-from-the-Sidebar
…ections-from-the-Sidebar
holy fuck how tf it passed pr lint... |
Read https://www.conventionalcommits.org/en/v1.0.0/ how this template works for commits for next time |
It passed because of the title I gave this “pull request”. If you notice, I even edited the title a few times because “it wasn't passing that ‘lint’”, so I thought it was a mistake. I experimented a bit and ended up reading what was written in a documentation that github suggested near the error to theoretically make it correct. I was initially going to put it as “feat(snippets): (...)” because of the other examples I've seen, but since I was getting an error with that “lint” and the documentation said it was just “feat:”, I ended up changing the terms. I honestly didn't know it was supposed to be any other way. I'm still relatively new to working with github, so sorry about that. I'm still learning a few things. This was my first attempt ahahahaha |
Addition of a new snippet that allows you to show the album cover and the next song centered at the same time. There is a similar snippet but it only shows the cover image. I prefer to have the next song also shown, so I've added this option as well.