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

More fixes for non-ASCII chars in MPD uris #1808

Merged
merged 1 commit into from Oct 2, 2019

Conversation

@kingosticks
Copy link
Member

kingosticks commented Oct 1, 2019

I missed Album URIs in #1805. And a changelog entry.

Related to #1805.
@kingosticks kingosticks force-pushed the kingosticks:fix/mpd-unicode-uri-2 branch from 4ab35dd to c59e798 Oct 1, 2019
@codecov

This comment has been minimized.

Copy link

codecov bot commented Oct 1, 2019

Codecov Report

Merging #1808 into release-2.3 will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@              Coverage Diff               @@
##           release-2.3   #1808      +/-   ##
==============================================
+ Coverage           81%     81%   +<.01%     
==============================================
  Files               83      83              
  Lines             6649    6650       +1     
==============================================
+ Hits              5386    5387       +1     
  Misses            1263    1263
Impacted Files Coverage Δ
mopidy/mpd/translator.py 95.4% <100%> (ø) ⬆️
mopidy/mpd/protocol/stored_playlists.py 90.96% <100%> (+0.05%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd47db4...c59e798. Read the comment docs.

@kingosticks kingosticks added the A-mpd label Oct 1, 2019
@kingosticks kingosticks added this to the v2.3.0 milestone Oct 1, 2019
@kingosticks kingosticks self-assigned this Oct 1, 2019
@jodal
jodal approved these changes Oct 2, 2019
@jodal jodal merged commit e2100a9 into mopidy:release-2.3 Oct 2, 2019
3 checks passed
3 checks passed
codecov/patch 100% of diff hit (target 81%)
Details
codecov/project 81% (+<.01%) compared to dd47db4
Details
test Workflow: test
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.