From 72a3a6fde774527651e61b1619efa0419582e590 Mon Sep 17 00:00:00 2001 From: silverwind Date: Tue, 26 Mar 2024 12:19:15 +0100 Subject: [PATCH] Add svg linter and fix incorrect svgs (#30086) Fixes https://github.com/go-gitea/gitea/issues/30082. Adds a new linter that searches for non-existant SVG images in templates. Output before the fix was: ``` $ make lint-templates SVG "octicon-warning" not found, used in templates/devtest/flex-list.tmpl SVG "octicon-warning" not found, used in templates/devtest/flex-list.tmpl SVG "octicon-markup" not found, used in templates/repo/diff/comment_form.tmpl make: *** [Makefile:438: lint-templates] Error 1 ``` Screenshot 2024-03-25 at 23 31 05 (cherry picked from commit 2ab5f05f40d93224f73e211e84de50a88a6ecf03) Conflicts: .github/workflows/files-changed.yml .github/workflows/pull-compliance.yml do not exist in Forgejo and the Forgejo workflows already contain the changes --- Makefile | 3 ++- templates/devtest/flex-list.tmpl | 4 ++-- templates/repo/diff/comment_form.tmpl | 2 +- tools/lint-templates-svg.js | 26 ++++++++++++++++++++++++++ 4 files changed, 31 insertions(+), 4 deletions(-) create mode 100755 tools/lint-templates-svg.js diff --git a/Makefile b/Makefile index 70a7f906e..8271067c2 100644 --- a/Makefile +++ b/Makefile @@ -458,7 +458,8 @@ lint-actions: $(GO) run $(ACTIONLINT_PACKAGE) .PHONY: lint-templates -lint-templates: .venv +lint-templates: .venv node_modules + @node tools/lint-templates-svg.js @poetry run djlint $(shell find templates -type f -iname '*.tmpl') .PHONY: lint-yaml diff --git a/templates/devtest/flex-list.tmpl b/templates/devtest/flex-list.tmpl index d5678566d..015ab1e15 100644 --- a/templates/devtest/flex-list.tmpl +++ b/templates/devtest/flex-list.tmpl @@ -25,7 +25,7 @@
diff --git a/templates/repo/diff/comment_form.tmpl b/templates/repo/diff/comment_form.tmpl index 6a5dec6c4..856b3da01 100644 --- a/templates/repo/diff/comment_form.tmpl +++ b/templates/repo/diff/comment_form.tmpl @@ -26,7 +26,7 @@ {{end}}