Support ignoring file deletions

Author: johnlk

README.md

@@ -72,6 +72,7 @@ jobs:
 | `github_api_url`        | No       | 'https://api.github.com' | URL for the GitHub API, can be changed for GitHub Enterprise Servers.                                                  |
 | `files_to_ignore`       | No       | ''                   | Files to ignore during PR size calculation. Supports newline or whitespace delimited list.                              |
 | `ignore_line_deletions` | No       | 'false'              | Whether to ignore lines which are deleted when calculating the PR size. If set to 'true', deleted lines will be ignored. |
+| `ignore_file_deletions` | No       | 'false'              | Whether to ignore completely deleted files when calculating the PR size. If set to 'true', deleted files will be ignored. |
 
 ### Example for `files_to_ignore`:
 ```yml
@@ -87,6 +88,7 @@ files_to_ignore: |
 
 - PR Size Labeler considers any line addition, deletion, or modification as a change.
 - A PR will be labeled as 'xl' if it exceeds the amount of changes defined in `l_max_size`.
+- `ignore_file_deletions` is distinct from `ignore_line_deletions` in that it only ignores files which are deleted completely.
 
 ## ⚖️ License
 

action.yml

@@ -63,6 +63,10 @@ inputs:
     description: 'Whether to ignore lines which are deleted when calculating the PR size. If set to "true", deleted lines will be ignored.'
     required: false
     default: 'false'
+  ignore_file_deletions:
+    description: 'Whether to ignore files which are deleted when calculating the PR size. If set to "true", deleted files will be ignored.'
+    required: false
+    default: 'false'
 runs:
   using: 'docker'
   image: 'Dockerfile'

src/github.sh

@@ -6,11 +6,12 @@ github::calculate_total_modifications() {
   local -r pr_number="${1}"
   local -r files_to_ignore="${2}"
   local -r ignore_line_deletions="${3}"
+  local -r ignore_file_deletions="${4}"
 
   local additions=0
   local deletions=0
 
-  if [ -z "$files_to_ignore" ]; then
+  if [ -z "$files_to_ignore" ] && [ "$ignore_file_deletions" != "true" ]; then
     local -r body=$(curl -sSL -H "Authorization: token $GITHUB_TOKEN" -H "$GITHUB_API_HEADER" "$GITHUB_API_URL/repos/$GITHUB_REPOSITORY/pulls/$pr_number")
 
     additions=$(echo "$body" | jq '.additions')
@@ -19,12 +20,18 @@ github::calculate_total_modifications() {
       ((deletions += $(echo "$body" | jq '.deletions')))
     fi
   else
+    # NOTE: this code is not resilient to changes w/ > 100 files as we're not paginating
     local -r body=$(curl -sSL -H "Authorization: token $GITHUB_TOKEN" -H "$GITHUB_API_HEADER" "$GITHUB_API_URL/repos/$GITHUB_REPOSITORY/pulls/$pr_number/files?per_page=100")
 
     for file in $(echo "$body" | jq -r '.[] | @base64'); do
       filename=$(jq::base64 '.filename')
+      status=$(jq::base64 '.status')
       ignore=false
 
+      if [ "$ignore_file_deletions" == "true" ] && [ "$status" == "removed" ]; then
+        continue
+      fi
+
       for pattern in $files_to_ignore; do
         if [[ $filename == $pattern ]]; then
           ignore=true

src/labeler.sh

@@ -10,9 +10,10 @@ labeler::label() {
   local -r message_if_xl="${11}"
   local -r files_to_ignore="${12}"
   local -r ignore_line_deletions="${13}"
+  local -r ignore_file_deletions="${14}"
 
   local -r pr_number=$(github_actions::get_pr_number)
-  local -r total_modifications=$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions")
+  local -r total_modifications=$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions" "$ignore_file_deletions")
 
   log::message "Total modifications (additions + deletions): $total_modifications"
   log::message "Ignoring files (if present): $files_to_ignore"

src/main.sh

@@ -9,7 +9,7 @@ source "$PR_SIZE_LABELER_HOME/src/misc.sh"
 ##? Adds a size label to a GitHub Pull Request
 ##?
 ##? Usage:
-##?   main.sh --github_token=<token> --xs_label=<label> --xs_max_size=<size> --s_label=<label> --s_max_size=<size> --m_label=<label> --m_max_size=<size> --l_label=<label> --l_max_size=<size> --xl_label=<label> --fail_if_xl=<false> --message_if_xl=<message> --github_api_url=<url> --files_to_ignore=<files> --ignore_line_deletions=<false>
+##?   main.sh --github_token=<token> --xs_label=<label> --xs_max_size=<size> --s_label=<label> --s_max_size=<size> --m_label=<label> --m_max_size=<size> --l_label=<label> --l_max_size=<size> --xl_label=<label> --fail_if_xl=<false> --message_if_xl=<message> --github_api_url=<url> --files_to_ignore=<files> --ignore_line_deletions=<false> --ignore_file_deletions=<false>
 main() {
   eval "$(/root/bin/docpars -h "$(grep "^##?" "$PR_SIZE_LABELER_HOME/src/main.sh" | cut -c 5-)" : "$@")"
 
@@ -32,7 +32,8 @@ main() {
     "$fail_if_xl" \
     "$message_if_xl" \
     "$files_to_ignore" \
-    "$ignore_line_deletions"
+    "$ignore_line_deletions" \
+    "$ignore_file_deletions"
 
   exit $?
 }

tests/fixtures/pull_request_files_api

@@ -1,74 +1,93 @@
 [
   {
     "filename": ".editorconfig",
+    "status": "added",
     "additions": 9,
     "deletions": 0,
     "changes": 9
   },
   {
     "filename": ".github/workflows/ci.yml",
+    "status": "modified",
     "additions": 17,
     "deletions": 0,
     "changes": 17
   },
   {
     "filename": "asdasdasd.lock",
+    "status": "modified",
     "additions": 1188,
     "deletions": 0,
     "changes": 1188
   },
   {
     "filename": "entrypoint.sh",
+    "status": "modified",
     "additions": 4,
     "deletions": 4,
     "changes": 8
   },
   {
     "filename": "src/ensure.sh",
+    "status": "modified",
     "additions": 4,
     "deletions": 4,
     "changes": 8
   },
   {
     "filename": "src/github.sh",
+    "status": "added",
     "additions": 89,
     "deletions": 76,
     "changes": 165
   },
   {
     "filename": "src/github_actions.sh",
+    "status": "modified",
     "additions": 1,
     "deletions": 1,
     "changes": 2
   },
   {
     "filename": "src/inner.lock",
+    "status": "modified",
     "additions": 1188,
     "deletions": 0,
     "changes": 1188
   },
   {
     "filename": "src/labeler.sh",
+    "status": "modified",
     "additions": 48,
     "deletions": 48,
     "changes": 96
   },
   {
     "filename": "src/log.sh",
+    "status": "modified",
     "additions": 17,
     "deletions": 0,
     "changes": 17
   },
   {
     "filename": "src/main.sh",
+    "status": "modified",
     "additions": 34,
     "deletions": 23,
     "changes": 57
   },
   {
     "filename": "src/misc.sh",
+    "status": "modified",
     "additions": 10,
     "deletions": 14,
     "changes": 24
+  },
+  {
+    "filename": "deleted_file.txt",
+    "status": "removed",
+    "additions": 0,
+    "deletions": 123,
+    "changes": 123
   }
 ]

tests/github_test.sh

@@ -8,36 +8,54 @@ function set_up() {
 pr_number=123
 files_to_ignore=''
 ignore_line_deletions='false'
+ignore_file_deletions='false'
 
 function test_should_count_changes() {
   mock curl cat ./tests/fixtures/pull_request_api
 
-  assert_match_snapshot "$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions")"
+  assert_match_snapshot "$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions" "$ignore_file_deletions")"
 }
 
-function test_should_count_changes_ignore_deletions() {
+function test_should_count_changes_ignore_line_deletions() {
   ignore_line_deletions='true'
 
   mock curl cat ./tests/fixtures/pull_request_api
 
-  assert_match_snapshot "$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions")"
+  assert_match_snapshot "$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions" "$ignore_file_deletions")"
 }
 
-# NOTE: when `files_to_ignore` is set, we have to invoke the PR files API and iterate each file
+# NOTE: when `files_to_ignore` or `ignore_file_deletions` is set, we have to invoke the PR files API and iterate each file
 # one at at time. This is why the mock call is diffent in the subsequent test cases
+function test_should_count_changes_ignore_file_deletions() {
+  ignore_file_deletions='true'
+
+  mock curl cat ./tests/fixtures/pull_request_files_api
+
+  assert_match_snapshot "$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions" "$ignore_file_deletions")"
+}
+
 function test_should_ignore_files_with_glob() {
   files_to_ignore=("*.lock" ".editorconfig")
 
   mock curl cat ./tests/fixtures/pull_request_files_api
 
-  assert_match_snapshot "$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions")"
+  assert_match_snapshot "$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions" "$ignore_file_deletions")"
 }
 
-function test_should_ignore_files_with_glob_ignore_deletions() {
+function test_should_ignore_files_with_glob_ignore_line_deletions() {
   files_to_ignore=("*.lock" ".editorconfig")
   ignore_line_deletions='true'
 
   mock curl cat ./tests/fixtures/pull_request_files_api
 
-  assert_match_snapshot "$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions")"
+  assert_match_snapshot "$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions" "$ignore_file_deletions")"
+}
+
+function test_should_ignore_files_with_glob_ignore_file_deletions() {
+  files_to_ignore=("*.lock" ".editorconfig")
+  ignore_file_deletions='true'
+
+  mock curl cat ./tests/fixtures/pull_request_files_api
+
+  assert_match_snapshot "$(github::calculate_total_modifications "$pr_number" "${files_to_ignore[*]}" "$ignore_line_deletions" "$ignore_file_deletions")"
 }

tests/snapshots/github_test_sh.test_should_count_changes_ignore_file_deletions.snapshot

@@ -0,0 +1 @@
+2779

…_count_changes_ignore_deletions.snapshot …t_changes_ignore_line_deletions.snapshottests/snapshots/github_test_sh.test_should_count_changes_ignore_deletions.snapshot renamed to tests/snapshots/github_test_sh.test_should_count_changes_ignore_line_deletions.snapshot tests/snapshots/github_test_sh.test_should_count_changes_ignore_deletions.snapshot renamed to tests/snapshots/github_test_sh.test_should_count_changes_ignore_line_deletions.snapshot

@@ -1 +1 @@
-394
+517