[CI] Add pre-commit hook checkmake to lint Makefiles

.pre-commit-config.yaml

@@ -117,6 +117,16 @@ repos:
           - --license-filepath
           - .github/workflows/license-templates/LICENSE.txt
           - --fuzzy-match-generates-todo
+      - id: insert-license
+        name: add license for all ini files
+        description: automatically adds a licence header to all ini files that don't have a license header
+        files: \.ini$
+        args:
+          - --comment-style
+          - '|;|'
+          - --license-filepath
+          - .github/workflows/license-templates/LICENSE.txt
+          - --fuzzy-match-generates-todo
       - id: insert-license
         name: add license for all Java files
         description: automatically adds a licence header to all Java files that don't have a license header
@@ -526,3 +536,16 @@ repos:
           - 'SC2102'
           - -ignore
           - 'SC2155'
+  - repo: https://github.com/checkmake/checkmake.git
+    rev: v0.3.2
+    hooks:
+      - id: checkmake
+        name: run checkmake (root)
+        description: checkmake is a linter for Makefiles
+        files: ^Makefile$
+        args: ['--config=checkmake.ini']
+      - id: checkmake
+        name: run checkmake (python/sedona/doc)
+        description: checkmake is a linter for Makefiles
+        files: ^python/sedona/doc/Makefile$
+        args: ['--config=python/sedona/doc/checkmake.ini']

Makefile

@@ -18,7 +18,7 @@
 PYTHON := $(shell command -v python || command -v python3 || echo python)
 PIP := $(PYTHON) -m pip
 
-.PHONY: check checkinstall checkupdate install docsinstall docsbuild clean test
+.PHONY: check checkinstall checkupdate install docsinstall docsbuild clean run-docs
 
 check:
 	@echo "Running pre-commit checks..."

checkmake.ini

@@ -0,0 +1,22 @@
+; Licensed to the Apache Software Foundation (ASF) under one
+; or more contributor license agreements.  See the NOTICE file
+; distributed with this work for additional information
+; regarding copyright ownership.  The ASF licenses this file
+; to you under the Apache License, Version 2.0 (the
+; "License"); you may not use this file except in compliance
+; with the License.  You may obtain a copy of the License at
+;
+;   http://www.apache.org/licenses/LICENSE-2.0
+;
+; Unless required by applicable law or agreed to in writing,
+; software distributed under the License is distributed on an
+; "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+; KIND, either express or implied.  See the License for the
+; specific language governing permissions and limitations
+; under the License.
+
+[maxbodylength]
+maxBodyLength = 10
+
+[minphony]
+required = check,checkinstall,checkupdate,install,docsinstall,docsbuild,clean,run-docs

python/sedona/doc/checkmake.ini

@@ -0,0 +1,22 @@
+; Licensed to the Apache Software Foundation (ASF) under one
+; or more contributor license agreements.  See the NOTICE file
+; distributed with this work for additional information
+; regarding copyright ownership.  The ASF licenses this file
+; to you under the Apache License, Version 2.0 (the
+; "License"); you may not use this file except in compliance
+; with the License.  You may obtain a copy of the License at
+;
+;   http://www.apache.org/licenses/LICENSE-2.0
+;
+; Unless required by applicable law or agreed to in writing,
+; software distributed under the License is distributed on an
+; "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+; KIND, either express or implied.  See the License for the
+; specific language governing permissions and limitations
+; under the License.
+
+[maxbodylength]
+maxBodyLength = 2
+
+[minphony]
+required = help,livehtml