Closed johanneswuerbach closed 1 week ago
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks :+1:
sentences/de/weather_HassGetWeather.yaml (1)
`5-6`: **Addition of a new sentence variant is appropriate.** The new sentence "wie ist[ das] Wetter" aligns with the existing structure and enhances the coverage of the `HassGetWeather` intent.tests/de/weather_HassGetWeather.yaml (1)
`3-8`: **New test cases are correctly formulated.** The new test cases for the sentence variants "wie ist das Wetter" and "wie ist Wetter" ensure that the `HassGetWeather` intent responds correctly. The responses are consistent with the existing tests.
@johanneswuerbach I could not fork your fork ;) So please find an implementation of the test here.
Add this to tests/de/weather_HassGetWeather.yaml
- sentences:
- "wie ist das Wetter"
- "wie ist Wetter"
intent:
name: HassGetWeather
response: 8 °C und regnerisch
Thanks folks, added the test.
@mib1185 could you please review this?
Align with the
en
version and add a whether sentence without a name.https://github.com/home-assistant/intents/blob/main/sentences/en/weather_HassGetWeather.yaml#L5-L6
Summary by CodeRabbit
New Features
HassGetWeather
intent in the German language.Tests
HassGetWeather
intent to handle generic weather inquiries.