Skip to content

nlbwmon: enhance init script with logging and health checks - #29709

Open
Mahmudul-009 wants to merge 3 commits into
openwrt:masterfrom
Mahmudul-009:patch-3
Open

nlbwmon: enhance init script with logging and health checks#29709
Mahmudul-009 wants to merge 3 commits into
openwrt:masterfrom
Mahmudul-009:patch-3

Merge branch 'master' into patch-3

e218ea5
Select commit
Loading
Failed to load commit list.
OpenWrt / FormalityCheck / Git & Commits failed Aug 13, 2026 in 0s

Git & Commits: Failed

Some commits do not follow the commit message guidelines — open the details below to see what to change.

Details

Checking PR #29709: nlbwmon: enhance init script with logging and health checks (Formalities Audit)

✅ Pull request originates from a dedicated feature branch (patch-3)

Commit 5332e70 - Enhance nlbwmon with logging, health checks, and options:

✅ Author and committer identities are valid
✅ Commit is not a merge commit
✅ Commit description lines adhere to width formatting rules
✅ Commit contains a valid 'Signed-off-by:' line matching author or committer
⚠️ Warning: No reference link (e.g., upstream release notes, changelog, or history URL) detected in description.
⚠️ Warning: Commit is unsigned or cryptographic signature verification failed (Reason: unsigned). Signing commits is a recommended best practice for verifying identity, but is not mandatory.
❌ Commit subject must start with <package name or prefix>:

Commit 85e2d43 - Refactor logging functions and improve error handling:

✅ Author and committer identities are valid
✅ Commit is not a merge commit
✅ Commit contains a valid 'Signed-off-by:' line matching author or committer
⚠️ Warning: No reference link (e.g., upstream release notes, changelog, or history URL) detected in description.
⚠️ Warning: Commit is unsigned or cryptographic signature verification failed (Reason: unsigned). Signing commits is a recommended best practice for verifying identity, but is not mandatory.
❌ Commit subject must start with <package name or prefix>:
❌ Line 3 in commit body exceeds max width (156/100 chars)

Commit e218ea5 - Merge branch 'master' into patch-3:

✅ Author identity is valid (committed through the GitHub web interface)
✅ Commit description lines adhere to width formatting rules
✅ Excellent! Commit contains a valid cryptographic signature (GPG/SSH). Thank you for signing your work! (Verified via GitHub Profile)
❌ Merge commits are not allowed within the pull request
❌ Commit subject must start with <package name or prefix>:
❌ Commit description body is empty or contains only trailers (e.g. Signed-off-by). Please provide a meaningful description of what this change does and why
❌ Missing 'Signed-off-by:' line. Please add a line at the end of the commit message in the format 'Signed-off-by: Your Name your@email.com', matching your commit author or committer identity.