Skip to content

Fix markdown bindings M-N #19025

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

Merged
merged 14 commits into from
Jul 26, 2025
Merged

Fix markdown bindings M-N #19025

merged 14 commits into from
Jul 26, 2025

Conversation

lsiepel
Copy link
Contributor

@lsiepel lsiepel commented Jul 23, 2025

Ref: #19011

Signed-off-by: Leo Siepel <[email protected]>
Copy link
Contributor

@andrewfg andrewfg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM NeoHub

Copy link
Contributor

@mlobstein mlobstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for monopriceaudio

@dag81
Copy link
Contributor

dag81 commented Jul 23, 2025

LGTM for metofficedatahub thank you

Copy link
Contributor

@ccutrer ccutrer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for mqtt.generic and mqtt.homeassistant

Signed-off-by: Leo Siepel <[email protected]>
Copy link
Contributor

@bern77 bern77 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for helioseasycontrols

Signed-off-by: Leo Siepel <[email protected]>
Copy link
Member

@wborn wborn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🆗 👌

@wborn wborn requested a review from Copilot July 26, 2025 09:15
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR addresses markdown formatting issues in various binding documentation files by fixing malformed tables, adding missing table delimiters, correcting link formats, and standardizing code block syntax highlighting. The changes improve documentation readability and maintain consistency across the openHAB binding documentation.

  • Fix table formatting issues including missing pipes and separators
  • Standardize URL references to use proper markdown link format
  • Add appropriate syntax highlighting to code blocks
  • Correct markdown heading levels and list formatting

Reviewed Changes

Copilot reviewed 30 out of 31 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
bundles/org.openhab.binding.nzwateralerts/README.md Fix missing pipe in table definition
bundles/org.openhab.binding.nibeheatpump/contrib/NibeGW/Arduino/README.md Fix heading levels and add syntax highlighting to code blocks
bundles/org.openhab.binding.nibeheatpump/README.md Add missing line break before table
bundles/org.openhab.binding.netatmo/README.md Add syntax highlighting and fix inline link format
bundles/org.openhab.binding.nest/README.md Correct URL angle bracket format
bundles/org.openhab.binding.neohub/README.md Add missing line break before table
bundles/org.openhab.binding.myuplink/README.md Add missing line break before Discovery section
bundles/org.openhab.binding.mybmw/README.md Fix image paths and URL references
bundles/org.openhab.binding.mqtt.homeassistant/README.md Remove extra blank line
bundles/org.openhab.binding.mqtt.homeassistant/DEVELOPMENT.md Add heading and improve list formatting
bundles/org.openhab.binding.mqtt.generic/README.md Add missing line break before table
bundles/org.openhab.binding.mqtt.fpp/README.md Add missing line break before Thing Configuration section
bundles/org.openhab.binding.mqtt.awtrixlight/README.md Correct URL angle bracket format
bundles/org.openhab.binding.monopriceaudio/README.md Fix italic text formatting
bundles/org.openhab.binding.modbus.sungrow/README.md Correct URL format and improve section headers
bundles/org.openhab.binding.modbus.kermi/README.md Remove extra blank lines
bundles/org.openhab.binding.modbus.helioseasycontrols/README.md Fix internal link references in table
bundles/org.openhab.binding.modbus.e3dc/README.md Fix spelling error, image alt text, and table formatting
bundles/org.openhab.binding.minecraft/README.md Add alt text to images
bundles/org.openhab.binding.miio/README.md Fix link text and remove extra blank line
bundles/org.openhab.binding.miio/README.base.md Fix link text reference
bundles/org.openhab.binding.metofficedatahub/README.md Fix table formatting issues
bundles/org.openhab.binding.meteostick/README.md Fix missing pipe in table
bundles/org.openhab.binding.meteofrance/README.md Convert bullet points to dashes, fix URL format, and add alt text
bundles/org.openhab.binding.meteoblue/README.md Fix table formatting and missing pipe
bundles/org.openhab.binding.meross/README.md Correct URL angle bracket format
bundles/org.openhab.binding.mercedesme/README.md Fix list numbering, table alignment, and text formatting
bundles/org.openhab.binding.matter/code-gen/README.md Improve list formatting and structure
bundles/org.openhab.binding.matter/README.md Remove blank lines, fix formatting, and improve structure
bundles/org.openhab.binding.matter/DEVELOPMENT.md Improve list numbering and formatting consistency

wborn and others added 2 commits July 26, 2025 20:50
…o/README.md

Co-authored-by: Jacob Laursen <[email protected]>
Signed-off-by: Wouter Born <[email protected]>
…o/README.md

Co-authored-by: Jacob Laursen <[email protected]>
Signed-off-by: Wouter Born <[email protected]>
@wborn wborn merged commit 0421d6e into openhab:main Jul 26, 2025
2 checks passed
@wborn wborn added this to the 5.1 milestone Jul 26, 2025
@lsiepel lsiepel deleted the fix/markdown-issues-m-n branch July 26, 2025 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants