new(ci): check PRs for semver violations - #76
Conversation
Signed-off-by: Grzegorz Nosek <grzegorz.nosek@sysdig.com>
e3fccda to
6632f6f
Compare
|
I'm wondering if it is possible to catch for PRs providing the |
Sure, just need to tweak the action a bit, though it... is... pretty... slow... (slower than a build+test run on linux) and I'm not sure it is all that beneficial, but I'm happy to be convinced (in any case, it's much faster than a full benchmark run) |
|
Yeah, not an issue. We can take it as is and introduce the check later if need arises |
|
LGTM label has been added. DetailsGit tree hash: 0bf3cebd1294d2640fbd340b39c445a23b501980 |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ekoops, gnosek The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area ci
What this PR does / why we need it:
(Try to) flag PRs which break the public API without an explicit
!marker in the titleWhich issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: