Feat/v14

.github/workflows/local-testing.yaml

@@ -11,19 +11,21 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Dev check PR
-        uses: JJ/github-pr-contains-action@dev
+        uses: JJ/github-pr-contains-action@feat/v14
         id: dev_check_pr
         with:
           github-token: ${{github.token}}
           bodyContains: "[x]"
           bodyDoesNotContain: "your|own"
-          diffDoesNotContain: "TODO"
-          linesChanged: 1
-          waivedUsers: "dependabot[bot]|JJ"
       - name: Info PR
-        with:
-          FILES : ${{ steps.dev_check_pr.outputs.files }}
-          DIFF : ${{ steps.dev_check_pr.outputs.diff }}
+        env:
+          NUMBER_OF_FILES : ${{ steps.dev_check_pr.outputs.numberOfFiles }}
+        run: |
+          echo "::warning::We got files: $NUMBER_OF_FILES"
+      - name: Fail if too big
+        if: ${{ steps.dev_check_pr.outputs.numberOfFiles > 3 }}
+        env:
+          NUMBER_OF_FILES : ${{ steps.dev_check_pr.outputs.numberOfFiles }}
         run: |
-          echo "We got files: $( echo -n $FILES | wc -l )"
-          echo "We got diff: $( echo -n $DIFF | wc -l ) lines in diff"
+          echo "::error::Too many files, there are $NUMBER_OF_FILES"
+          exit 1

action.yml

@@ -24,8 +24,8 @@ inputs:
 outputs:
   diff:
     description: "Lines added to the PR"
-  files:
-    description: "Files changed in the PR"
+  numberOfFiles:
+    description: "Number of files changed in the PR"
 
 runs:
   using: "node20"

dist/index.js

@@ -126,15 +126,15 @@ function run() {
                 core.info("Checking diff contents");
                 const diffContains = core.getInput("diffContains");
                 const diffDoesNotContain = core.getInput("diffDoesNotContain");
-                const files = yield getDiff(octokit, repository, pull_request);
-                core.setOutput("files", files);
+                const parsedDiff = yield getDiff(octokit, repository, pull_request);
+                core.setOutput("numberOfFiles", parsedDiff.length);
                 const filesChanged = +core.getInput("filesChanged");
-                if (filesChanged && files.length != filesChanged) {
+                if (filesChanged && parsedDiff.length != filesChanged) {
                     core.setFailed("You should change exactly " + filesChanged + " file(s)");
                 }
                 let changes = "";
                 let additions = 0;
-                files.forEach(function (file) {
+                parsedDiff.forEach(function (file) {
                     additions += file.additions;
                     file.chunks.forEach(function (chunk) {
                         chunk.changes.forEach(function (change) {

src/main.ts

@@ -92,18 +92,18 @@ async function run() {
       const diffContains = core.getInput("diffContains");
       const diffDoesNotContain = core.getInput("diffDoesNotContain");
 
-      const files = await getDiff(octokit, repository, pull_request);
-      core.setOutput("files", files);
+      const parsedDiff = await getDiff(octokit, repository, pull_request);
+      core.setOutput("numberOfFiles", parsedDiff.length);
       const filesChanged = +core.getInput("filesChanged");
-      if (filesChanged && files.length != filesChanged) {
+      if (filesChanged && parsedDiff.length != filesChanged) {
         core.setFailed(
           "You should change exactly " + filesChanged + " file(s)"
         );
       }
 
       let changes = "";
       let additions: number = 0;
-      files.forEach(function (file) {
+      parsedDiff.forEach(function (file) {
         additions += file.additions;
         file.chunks.forEach(function (chunk: parse.Chunk) {
           chunk.changes.forEach(function (change: any) {