pr-should-include-tests: no more CI:DOCS override

[edsantiago]

CI:DOCS is no more.

Signed-off-by: Ed Santiago santiago@redhat.com

None

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: edsantiago

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [edsantiago]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[packit-as-a-service]

Ephemeral COPR build failed. @containers/packit-build please check.

[Luap99]

Should we redefine the rules to better match our cirrus rules?
I think we could easily say unless a .go or .c was changed and not only in vendor/ or test/ then we do not need a test. To me this would help here to simply file the large
filtered_changes list

[edsantiago]

Possibly. I would need a thorough review of exceptions, such as .conf files, or pkg/*/json-and-yaml-and-zip, and probably add .s even though there are no such files today.

Also, in the process of looking into this, I realized you need to add *.h to your changesInclude()

I'm OOTO today, will think about it more tomorrow. Thanks for the suggestion.

[Luap99]

I realized you need to add *.h to your changesInclude()

I didn't though about .h because I thought all the symbol definitions are done in the go file in the special c comments but looks like you are right we have one .h file. I spin up a PR to add it.

[mheon]

This PR LGTM

[lsm5]

/packit rebuild

[baude]

/lgtm