From 6764fe03d0357be6afbe7c2b554df187cdcb7206 Mon Sep 17 00:00:00 2001 From: Ed Santiago Date: Mon, 25 Jul 2022 09:00:36 -0600 Subject: CI: new check for leftover skips/fixmes If a PR says "Fixes #123", make sure it removes skips and/or FIXME comments that reference issue 123. Signed-off-by: Ed Santiago --- Makefile | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'Makefile') diff --git a/Makefile b/Makefile index d172f2048..ca4b17d8f 100644 --- a/Makefile +++ b/Makefile @@ -264,7 +264,7 @@ codespell: codespell -S bin,vendor,.git,go.sum,.cirrus.yml,"RELEASE_NOTES.md,*.xz,*.gz,*.ps1,*.tar,swagger.yaml,*.tgz,bin2img,*ico,*.png,*.1,*.5,copyimg,*.orig,apidoc.go" -L pullrequest,uint,iff,od,seeked,splitted,marge,erro,hist,ether -w .PHONY: validate -validate: lint .gitvalidation validate.completions man-page-check swagger-check tests-included tests-expect-exit +validate: lint .gitvalidation validate.completions man-page-check swagger-check tests-included tests-expect-exit pr-removes-fixed-skips .PHONY: build-all-new-commits build-all-new-commits: @@ -621,6 +621,10 @@ tests-expect-exit: exit 1; \ fi +.PHONY: pr-removes-fixed-skips +pr-removes-fixed-skips: + contrib/cirrus/pr-removes-fixed-skips + ### ### Release/Packaging targets ### -- cgit v1.2.3-54-g00ecf