Skip to content

New tests

New tests #384

Triggered via pull request February 6, 2024 11:14
Status Failure
Total duration 41s
Artifacts

commitlint.yml

on: pull_request
Check if the commits meet the requirements of the guidelines
32s
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: zebra: Show IPv6 link-local interfaces under `show interface brief` output Despite if it's managed by FRR or the kernel, show it. If the system has only link-local addresses, we should show it unless it's a secondary one. Signed-off-by: Donatas Abraitis <[email protected]> ✖ header must not be longer than 72 characters, current length is 74 [header-max-length] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: redhat: rpm spec: added protobuf-c to deps & fix bogus changelog date Fixes for: ``` checking for protoc-c... no configure: error: in `/builddir/build/BUILD/frr-frr-9.1': configure: error: protobuf requested but protoc-c not found. Install protobuf-c. ``` And rpmlint warnings: ``` warning: line 208: It's not recommended to have unversioned Obsoletes: Obsoletes: gated mrt zebra frr-sysvinit warning: bogus date in %changelog: Thu Oct 09 2023 Donatas Abraitis <[email protected]> - 9.1 line 208: It's not recommended to have unversioned Obsoletes: Obsoletes: gated mrt zebra frr-sysvinit bogus date in %changelog: Thu Oct 09 2023 Donatas Abraitis <[email protected]> - 9.1 ``` Signed-off-by: Konstantin Shalygin <[email protected]> ✖ type must be one of [babeld, bfdd, bgpd, build, doc, docker, eigrpd, fpm, isisd, ldpd, lib, mgmtd, multi, nhrpd, ospf6d, ospfd, pathd, pbrd, pimd, pim6d, ripd, ripngd, sharpd, staticd, tests, tools, vtysh, vrrpd, yang, zebra, all] [type-enum] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: github: Mark the build as failed if 'do not merge' label is set This prevents us from merging PRs marked as 'do not merge'. Signed-off-by: Donatas Abraitis <[email protected]> ✖ type must be one of [babeld, bfdd, bgpd, build, doc, docker, eigrpd, fpm, isisd, ldpd, lib, mgmtd, multi, nhrpd, ospf6d, ospfd, pathd, pbrd, pimd, pim6d, ripd, ripngd, sharpd, staticd, tests, tools, vtysh, vrrpd, yang, zebra, all] [type-enum] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ 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
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/.