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

Add yamaha_musiccast sound modes #30880

Closed

Conversation

uvjustin
Copy link
Contributor

Description:

Added sound modes to yamaha-musiccast component. @jalmeroth

Checklist:

  • [x ] The code change is tested and works locally.

@homeassistant
Copy link
Contributor

Hi @uvjustin,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

@probot-home-assistant
Copy link

Hey there @jalmeroth, mind taking a look at this pull request as its been labeled with a integration (yamaha_musiccast) you are listed as a codeowner for? Thanks!

@MartinHjelmare MartinHjelmare changed the title Added sound modes Add yamaha-musiccast sound modes Jan 17, 2020
@MartinHjelmare MartinHjelmare changed the title Add yamaha-musiccast sound modes Add yamaha_musiccast sound modes Jan 17, 2020
@springstan
Copy link
Member

@uvjustin please solve all CI errors by following the development checklist:)

@uvjustin
Copy link
Contributor Author

Just ran black (it corrected import order and added newline at the end). However, the change here was minimal - most of the changes are actually in the PYPI library, and the requirements won't be satisfied until that package is reviewed/updated by the maintainer.
I'm new to Github so let me know if you have any pointers about how to contribute/collaborate. (I have a contribution to a different integration ready, but I'll wait to push it until I get more familiar with this process.)

@frenck
Copy link
Member

frenck commented Jan 21, 2020

@uvjustin From this end, it looks like the formatting with black is the only apparent issue for the CI to pass at this point. Format it, commit it, push it, and see how far the build will go after that. 👍

@springstan
Copy link
Member

@uvjustin any updates on this? :)

@uvjustin
Copy link
Contributor Author

uvjustin commented Feb 2, 2020

@springstan
I'm not the package maintainer of the pypi package so we'll have to wait for my pull request to get merged on that side.

@springstan
Copy link
Member

@uvjustin please link the PR that needs to be merged in the python package and please format your code with black and push it so we can take a look at it :)

@uvjustin
Copy link
Contributor Author

uvjustin commented Feb 2, 2020

@springstan Here is the PR jalmeroth/pymusiccast#15

@springstan springstan added the waiting-for-upstream We're waiting for a change upstream label Feb 2, 2020
@stale
Copy link

stale bot commented Mar 21, 2020

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days.
Thank you for your contributions.

@stale stale bot added the stale label Mar 21, 2020
@uvjustin
Copy link
Contributor Author

Ok with closing this. I will continue to use my own version locally for the sound modes functionality. If there's more interest in updating the pymusiccast library I can fork it and update the HA core to use that library instead.

@stale stale bot removed the stale label Mar 24, 2020
@MartinHjelmare
Copy link
Member

I'll close here since this is still waiting for upstream.

@TheZoker
Copy link
Contributor

@uvjustin please fork the library and redo this PR. I have two Yamaha musiccast speakers and am waiting for this PR

@lock lock bot locked and limited conversation to collaborators Apr 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-signed integration: yamaha_musiccast small-pr PRs with less than 30 lines. waiting-for-upstream We're waiting for a change upstream
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants