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

fix: Make dashcast work with audio devices #107

Merged
merged 2 commits into from
May 31, 2018
Merged

Conversation

theychx
Copy link
Collaborator

@theychx theychx commented May 31, 2018

This might sound like a silly idea, but it could be useful as a fallback for sites with autoplaying content that is not supported by youtube-dl.

@theychx
Copy link
Collaborator Author

theychx commented May 31, 2018

I don't understand the CI failure. The hook ran fine when commiting.

@skorokithakis
Copy link
Owner

Sorry, that one was on me. I forgot to include it in the dev requirements.

@skorokithakis
Copy link
Owner

Hm, this is interesting. Have you tried to see if it works? I'm currently having issues discovering my Mi box at all.

@theychx
Copy link
Collaborator Author

theychx commented May 31, 2018

Yep, works with soundcloud.

@theychx
Copy link
Collaborator Author

theychx commented May 31, 2018

CI fails now for py <3.6 because of black.

@skorokithakis
Copy link
Owner

Yep, that's super inconvenient...

@skorokithakis skorokithakis merged commit 953f383 into master May 31, 2018
@theychx
Copy link
Collaborator Author

theychx commented May 31, 2018

How did you resolve this?

@theychx theychx deleted the audio-dashcast branch May 31, 2018 14:34
@skorokithakis
Copy link
Owner

skorokithakis commented May 31, 2018

I just force-merged, I'll resolve later as I'm very busy now :(

@theychx
Copy link
Collaborator Author

theychx commented May 31, 2018

I have some PR's coming up, most of them small. Is there any specific time you'd prefer to deal with this? I can also roll some of the small changes into a couple of PR's, if you prefer.

@skorokithakis
Copy link
Owner

Nah just send them all in and I'll get to them as I can.

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.

2 participants