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

plugins.vk: _url_re update, allow embedded content, plugin cleanup #1874

Merged
merged 2 commits into from
Jun 28, 2018
Merged

Conversation

back-to
Copy link
Collaborator

@back-to back-to commented Jun 28, 2018

  • _url_re update
  • allow embedded content
  • plugin cleanup / use API

Added some itertags tests


close #1584
close #1603

@codecov
Copy link

codecov bot commented Jun 28, 2018

Codecov Report

Merging #1874 into master will increase coverage by <.01%.
The diff coverage is 24.39%.

@@            Coverage Diff             @@
##           master    #1874      +/-   ##
==========================================
+ Coverage   50.51%   50.52%   +<.01%     
==========================================
  Files         238      238              
  Lines       14077    14073       -4     
==========================================
- Hits         7111     7110       -1     
+ Misses       6966     6963       -3

@beardypig
Copy link
Member

Doh I had some itertags tests here https://github.com/beardypig/streamlink/blob/1e5cecb2591f520bad5ddb54c873d2361491be0b/tests/test_plugin_utils.py, but forgot to include them...

@back-to
Copy link
Collaborator Author

back-to commented Jun 28, 2018

they are in #1675

well they look better, will remove mine.

@back-to back-to added the plugin issue A Plugin does not work correctly label Jun 28, 2018
@gravyboat
Copy link
Member

Looks good @back-to, thanks!

@gravyboat gravyboat merged commit cbfe155 into streamlink:master Jun 28, 2018
@back-to back-to deleted the vk branch July 3, 2018 21:08
beardypig pushed a commit to beardypig/streamlink that referenced this pull request Jul 25, 2018
…treamlink#1874)

* plugins.vk: _url_re update, allow embedded content, plugin cleanup

* tests.utils: Removed itertags tests in favor of another PR.
mkbloke pushed a commit to mkbloke/streamlink that referenced this pull request Aug 18, 2020
…treamlink#1874)

* plugins.vk: _url_re update, allow embedded content, plugin cleanup

* tests.utils: Removed itertags tests in favor of another PR.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin issue A Plugin does not work correctly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

VK VoD play is broken
3 participants