Skip to content

New tests

New tests #407

Triggered via pull request February 20, 2024 01:55
Status Failure
Total duration 40s
Artifacts

commitlint.yml

on: pull_request
Check if the commits meet the requirements of the guidelines
31s
Check if the commits meet the requirements of the guidelines
Fit to window
Zoom out
Zoom in

Annotations

1 error and 1 warning
Check if the commits meet the requirements of the guidelines
You have commit messages with errors ⧗ input: doc: add diagram of oper-state queries. Signed-off-by: Christian Hopps <[email protected]> ✖ subject may not end with full stop [subject-full-stop] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: *: Remove Crypto openSSL define from zebra.h Only a couple of places use this. Move these to a better spot. Signed-off-by: Donald Sharp <[email protected]> ✖ type may not be empty [type-empty] ✖ subject may not be empty [subject-empty] ✖ found 2 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: lib: Move some priviledge headers to file they are used. Signed-off-by: Donald Sharp <[email protected]> ✖ subject may not end with full stop [subject-full-stop] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: *: Remove cost of including netinet/icmp.h everywhere This file inclusion is only used in 3 places move the includes to those spots. Signed-off-by: Donald Sharp <[email protected]> ✖ type may not be empty [type-empty] ✖ subject may not be empty [subject-empty] ✖ found 2 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: lib, vtysh: Remove inclusion of grp.h for everyone Signed-off-by: Donald Sharp <[email protected]> ✖ type may not be empty [type-empty] ✖ subject may not be empty [subject-empty] ✖ found 2 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: *: Remove sys/ioctl.h from zebra.h Practically no-one uses this and ioctls are pretty much wrappered. Further wrappering could make this even better. Signed-off-by: Donald Sharp <[email protected]> ✖ type may not be empty [type-empty] ✖ subject may not be empty [subject-empty] ✖ found 2 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: lib, sharpd: Move sys/resource.h to where it is used Signed-off-by: Donald Sharp <[email protected]> ✖ type may not be empty [type-empty] ✖ subject may not be empty [subject-empty] ✖ found 2 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: *: sys/uio.h does not need to be in zebra.h Signed-off-by: Donald Sharp <[email protected]> ✖ type may not be empty [type-empty] ✖ subject may not be empty [subject-empty] ✖ found 2 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: lib: fix initializer warning on centos 7. Signed-off-by: Christian Hopps <[email protected]> ✖ subject may not end with full stop [subject-full-stop] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint
Check if the commits meet the requirements of the guidelines
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.