arjav1528 opened a new pull request, #18426: URL: https://github.com/apache/nuttx/pull/18426
## Fixes #17133 ## Summary checkpatch.sh didnt require a blank line after the commit subject, so the length check could be bypassed by continuing the subject on line 2. The current changes require a blank second line and adds optional `-b` breaking-change checks (`!` in subject, `BREAKING CHANGE:` before sign-off). CI runs checkpatch with `-b` when the PR has the "breaking change" label. `--stdin` with `-m -g` allows message-only checks from stdin. ## Impact Commits that continued the subject on line 2 will now fail until a blank line is added. PRs with the "breaking change" label are validated in CI for the correct format. No build, hardware, or documentation impact. ## Testing Verified on macOS with `./tools/checkpatch.sh -m -g --stdin`: bypass case is rejected, valid messages pass, and `-b` correctly enforces breaking-change format. Normal `-m -g HEAD` path and CI label logic were checked. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
