Merge pull request #214 from gibbonCode/code-issues-action

CodeIssuesTask action

Author: Kevin-Mattheus-Moerman

.github/workflows/run_tests.yml

@@ -41,6 +41,9 @@ jobs:
         with:
           command: installGibbon
 
+      - name: Check Code-Issues
+        uses: matlab-actions/run-build@v3
+
       - name: Run tests
         uses: matlab-actions/run-tests@v3
         with:
@@ -56,4 +59,4 @@ jobs:
           files: ./code-coverage/coverage.xml
           flags: matlab
           token: ${{ secrets.CODECOV_TOKEN }}
-          fail_ci_if_error: false
+          fail_ci_if_error: false

buildfile.m

@@ -0,0 +1,9 @@
+function plan = buildfile
+import matlab.buildtool.tasks.*
+
+plan = buildplan(localfunctions);
+
+fc = matlab.buildtool.io.FileCollection.fromPaths(["docs/*.m", "lib/*.m", "tests/*.m"]);
+plan("check") = CodeIssuesTask(fc);
+plan.DefaultTasks = "check";
+end

docs/DEMO_febio_0059_face_mask_loading.m

@@ -536,7 +536,7 @@
 
 [Fr1,Vr1]=roundMesh(indStart_Vm,Vm,Nm,nRim,maskRimFilletRadius);
 [Fr2,Vr2]=roundMesh(indEnd_Vm,Vm,Nm,nRim,maskRimFilletRadius);
-indEnd_Vr1=size(Vr1)-numPointsRimCurve+1:1:size(Vr1);
+indEnd_Vr1=size(Vr1,1) - numPointsRimCurve + 1:size(Vr1,1);
 indEnd_Vr2=fliplr(indEnd_Vr1);
 [Fr1,Vr1]=quad2tri(Fr1,Vr1,'a');
 [Fr2,Vr2]=quad2tri(Fr2,Vr2,'a');

resources/codeAnalyzerConfiguration.json

@@ -0,0 +1,23 @@
+// https://uk.mathworks.com/help/matlab/matlab_env/configure-code-analyzer.html
+// https://uk.mathworks.com/help/matlab/matlab_env/index-of-code-analyzer-checks.html
+{
+    "name": "GIBBON Guideline",
+    "guidelineVersion": "0.0.0",
+    "baseConfiguration": "factory",
+
+    "checks": {
+        "JAVFM": {
+            // JavaFrame was undocummented and has been removed
+            "severity": "warning"
+        },
+        "DisallowFindJObj": {
+            "rule": {
+                "template": "functionCall",
+                "functionNames": "findjobj"
+            },
+            "severity": "warning",
+            "messageText": "findjobj relies on JavaFrame, removed in R2025a",
+            "enabled": true
+        }
+    },
+}