Fix documentation, grammar, and spelling in bindings U-V#19602
Fix documentation, grammar, and spelling in bindings U-V#19602jlaur merged 11 commits intoopenhab:mainfrom
Conversation
|
Failing build is not related to this PR. See #19584 |
|
Failing build is not related. Please merge to prevent further conflicts. |
There was a problem hiding this comment.
Pull request overview
This PR systematically improves documentation quality across multiple openHAB bindings in the U-V alphabetical range. It focuses on correcting spelling errors, improving grammar, standardizing terminology, and enhancing clarity throughout README files, properties files, and XML configuration files.
Key improvements include:
- Standardizing capitalization (e.g., "Thing" vs "thing", "Wi-Fi" vs "WiFi", "OLED" vs "Oled")
- Fixing spelling errors (e.g., "Refresj" → "Refresh", "SpringSteen" → "Springsteen", "ciruit" → "circuit")
- Improving grammar and sentence structure throughout documentation
- Correcting table formatting and alignment issues
Reviewed changes
Copilot reviewed 45 out of 46 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| bundles/org.openhab.binding.volvooncall/* | Fixed comment capitalization and table alignment |
| bundles/org.openhab.binding.volumio/README.md | Fixed spacing and grammar in channel descriptions |
| bundles/org.openhab.binding.vizio/README.md | Standardized "Thing" capitalization |
| bundles/org.openhab.binding.vitotronic/* | Added missing max values, fixed spelling ("ciruit" → "circuit"), improved grammar |
| bundles/org.openhab.binding.visualcrossing/README.md | Improved description clarity and conciseness |
| bundles/org.openhab.binding.vigicrues/README.md | Standardized Thing capitalization, fixed "tower" → "toward" |
| bundles/org.openhab.binding.vesync/* | Added outlet configuration and channels |
| bundles/org.openhab.binding.verisure/* | Fixed duplicate option values, improved table alignment |
| bundles/org.openhab.binding.ventaair/README.md | Standardized Thing terminology and improved table formatting |
| bundles/org.openhab.binding.venstarthermostat/* | Improved description clarity and consistency |
| bundles/org.openhab.binding.velux/* | Reorganized properties file sections, improved clarity |
| bundles/org.openhab.binding.velbus/* | Fixed "OLED" capitalization, corrected label ("VMPVP1" → "VMBVP1"), improved descriptions |
| bundles/org.openhab.binding.vektiva/* | Improved configuration descriptions and label consistency |
| bundles/org.openhab.binding.vdr/* | Changed "Sub Title" → "Subtitle" throughout |
| bundles/org.openhab.binding.valloxmv/* | Improved descriptions, corrected max value (1 → 100), added unit specifications |
| bundles/org.openhab.binding.urtsi/* | Added missing "required" attribute, added unit specification |
| bundles/org.openhab.binding.upnpcontrol/* | Fixed spelling, improved descriptions, reorganized properties file |
| bundles/org.openhab.binding.upb/README.md | Improved clarity and grammar throughout |
| bundles/org.openhab.binding.unifiedremote/* | Fixed incorrect channel reference, improved descriptions |
| bundles/org.openhab.binding.unifi/* | Improved clarity, fixed grammar, standardized terminology |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jlaur
left a comment
There was a problem hiding this comment.
Thanks! I added a few comments.
bundles/org.openhab.binding.unifiedremote/src/main/resources/OH-INF/thing/thing-types.xml
Outdated
Show resolved
Hide resolved
bundles/org.openhab.binding.valloxmv/src/main/resources/OH-INF/thing/thing-types.xml
Show resolved
Hide resolved
bundles/org.openhab.binding.vitotronic/src/main/resources/OH-INF/thing/thing-types.xml
Show resolved
Hide resolved
Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Signed-off-by: Leo Siepel <leosiepel@gmail.com>
Signed-off-by: Leo Siepel <leosiepel@gmail.com>
|
@Rosen01 would be appreciated if you double check the changes to the readme.md for the velbus binding. I had to fix some merge conflicts and improve at the same time. |
jlaur
left a comment
There was a problem hiding this comment.
Waiting for response to #19602 (comment). LGTM.
bundles/org.openhab.binding.velbus/src/main/resources/OH-INF/thing/channel-types.xml
Outdated
Show resolved
Hide resolved
…hing/channel-types.xml Signed-off-by: lsiepel <leosiepel@gmail.com>
No description provided.