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

Artist songs fixes #1284

Merged
merged 13 commits into from
May 24, 2021
Merged

Artist songs fixes #1284

merged 13 commits into from
May 24, 2021

Conversation

xnetcat
Copy link
Member

@xnetcat xnetcat commented May 10, 2021

Title

Artist songs fixes

Description

Fixed duplicates when getting songs from artist link

Related Issue

image

Motivation and Context

https://discord.com/channels/771628785447337985/796571887635267614/837733338927202344

How Has This Been Tested?

tests

Screenshots (if appropriate)

image

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project
  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have read the CONTRIBUTING document
  • I have read the CORE VALUES document
  • I have added tests to cover my changes
  • All new and existing tests passed

@xnetcat xnetcat requested review from Silverarmor and a user May 10, 2021 07:02
@xnetcat xnetcat added Bug Fix PRs that fix bugs Enhancement Enhancing spotDL labels May 10, 2021
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good. A little functional de-nesting and contextualisation required. Overall, pretty neat.

spotdl/search/utils.py Show resolved Hide resolved
spotdl/search/utils.py Outdated Show resolved Hide resolved
spotdl/search/utils.py Show resolved Hide resolved
spotdl/search/utils.py Outdated Show resolved Hide resolved
spotdl/search/utils.py Show resolved Hide resolved
@xnetcat xnetcat requested a review from a user May 17, 2021 13:56
@Silverarmor
Copy link
Member

  1. @MikhailZex Please re-review, you have requested changes.
  2. @xnetcat Please pull from upstream/dev

@Silverarmor
Copy link
Member

@xnetcat is this ready to go?

@xnetcat
Copy link
Member Author

xnetcat commented May 24, 2021

yup it's ready

@Silverarmor Silverarmor merged commit 740830f into spotDL:dev May 24, 2021
@xnetcat xnetcat deleted the artist-songs-fixes branch May 24, 2021 22:12
Silverarmor added a commit that referenced this pull request May 25, 2021
Publish v3.6.0

* ignore .cache and other hidden files (#1274)

* Bump minimal required python version to 3.6.1 (#1278)

* Remove FFmpeg normalization causing "quiet" songs. (#1276)

* Saved Songs Download and User Authentication (#1240)

* regenerate cassettes (#1290)

* Use ffmpeg_path to check for version (#1289)

* Skip already downloaded songs before doing youtube search (#1287)

* Fix security risk (#1285)

* Song matching improvements (#1279)

* Artist songs fixes (#1284)

* More output formats (#1244)

* Bump version number to 3.6.0

* Update .gitignore to remove duplicate cache

* docs update (#1293)


Co-authored-by: Silverarmor <23619946+Silverarmor@users.noreply.github.com>
Co-authored-by: Jakub Kot <42355410+xnetcat@users.noreply.github.com>
Co-authored-by: Peyton Creery <44987569+phcreery@users.noreply.github.com>
Co-authored-by: AZMCode <adrianozambrana@protonmail.com>
Co-authored-by: Aiden Gardner <19619206+aiden2480@users.noreply.github.com>
Co-authored-by: Oliver Blanthorn <freedom4cows@gmail.com>
Co-authored-by: Andrzej Klajnert <github@aklajnert.pl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Fix PRs that fix bugs Enhancement Enhancing spotDL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants