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

Publish 3.9 #1393

Merged
merged 12 commits into from
Oct 24, 2021
Merged

Publish 3.9 #1393

merged 12 commits into from
Oct 24, 2021

Conversation

Silverarmor
Copy link
Member

Title

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate)

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 and others added 9 commits September 26, 2021 23:07
* 🪶: Refactor Code Expression (#1)

* 🪶: Refactor Code Expression

* 🪶: Refactor Code Expression

* 🪶: Refactor Code Expression

* Fix Linting Issue (Based on mypy & Flake8)

* Using Black to Reformat Projects ✨(#2)

* Revert Comments ✨ (#3)

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>
* bugfix: replacez rapidfuzz with thefuzz

* misc: run ci on python3.10
* Add test coverage file to gitignore

Also remove old .cache

* Change tests to run with 1 search thread
* misc: removed gathering spotify metadata message

* misc: removed debug termination

* misc: bump default search threads to 4 for cli

* Revert "misc: removed debug termination"

This reverts commit 5787bea.

Co-authored-by: Silverarmor <23619946+Silverarmor@users.noreply.github.com>
Co-authored-by: Silverarmor <23619946+Silverarmor@users.noreply.github.com>
Co-authored-by: Jakub Kot <xnetcat.dev@gmail.com>
* misc: fix warnings

* misc: fix spotipy warning
@Silverarmor Silverarmor added the Skip Changelog PRs to skip changelog label Oct 23, 2021
@Silverarmor Silverarmor self-assigned this Oct 23, 2021
Silverarmor and others added 3 commits October 23, 2021 13:26
thefuzz was giving a headache. Revert #1391 back to rapidfuzz.
* ci: quote python version numbers

Future proofes for coming 3.10.X versions by chaning 3.10.0 to "3.10"

* ci: remove max parallel

Allow github to maximise number of parallel jobs
* feat: added path template

added playlist

added some more checks

simplified code

black

default to None

Update provider_utils.py

Update downloader.py

cassettes

cassettes again

Update query_parser.py

* feat: Add short flag -p  and edit help message and readme

Co-authored-by: Silverarmor <23619946+Silverarmor@users.noreply.github.com>
@Silverarmor Silverarmor merged commit 8474fee into master Oct 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs to skip changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants