Skip to content

Slight fix to "effects" #19288

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.

Sign up for GitHub

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

Jump to bottom
Closed
jds11111 opened this issue Sep 10, 2021 · 1 comment · Fixed by #19497
Closed

Slight fix to "effects" #19288

jds11111 opened this issue Sep 10, 2021 · 1 comment · Fixed by #19497
Labels
integration: marytts

Comments

@jds11111
Copy link

Feedback

I don't believe that the commas should be there after each of the effects. My Visual Studo editor complained about them. At any rate, removing them passes the HA configuration check, and it works fine without them.

URL

https://www.home-assistant.io/integrations/marytts/

Version

2021.9.5

Additional information

No response
@frenck frenck mentioned this issue Sep 27, 2021
Merged
8 tasks
@frenck
Copy link
Member

frenck commented Sep 27, 2021

Thanks for reporting this one @jds11111 👍

I've opened up a PR to adjust this

@github-actions github-actions bot locked and limited conversation to collaborators Nov 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
integration: marytts
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix MaryTTS example YAML home-assistant/home-assistant.io
2 participants
@frenck @jds11111