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

Choose 'Highest' audio quality by default #150

Merged
merged 2 commits into from
Aug 19, 2024
Merged

Conversation

nick42d
Copy link
Owner

@nick42d nick42d commented Aug 19, 2024

Closes #143

@nick42d nick42d linked an issue Aug 19, 2024 that may be closed by this pull request
@nick42d nick42d mentioned this pull request Aug 19, 2024
@nick42d nick42d merged commit 1adbd73 into main Aug 19, 2024
3 checks passed
@nick42d nick42d deleted the iss143-test-audio-quality branch August 19, 2024 12:23
@github-actions github-actions bot mentioned this pull request Aug 19, 2024
nick42d added a commit that referenced this pull request Aug 19, 2024
* Refactor common module
* fix: Make the channel/artist thumbnail on playlists/albums optional (#147)
* fix: Choose 'Highest' audio quality by default (#150) - resolves #143
* Increase audio quality - possibly resolves crackling
* Choose HighestAudio quality by default
* Complete marking public structs non exhaustive and implementing derives. Also, fixed two integration test issues; search videos failing due to a non-available song, and a flake issue on add/remove history
* Rename some awkward public structs, fix doctests
* Refactor common module
* Complete marking public structs non exhaustive and implementing derives. Also, fixed two integration test issues; search videos failing due to a non-available song, and a flake issue on add/remove history
* Rename some awkward public structs, fix doctests
* Fix failing test; renamed struct

BREAKING CHANGE: This is a significant breaking change, primarily due to  marking many structs non_exhaustive. This breakage now will save breakage in the future. In addition, significant refactoring between modules was undertaken to better organise the project.  Further to this, a small number of structs were renamed to better indicate their purpose.
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.

Noise during playback
1 participant