Report list of modified upgrade.txt or UPGRADING.md files

Signed-off-by: Daniel Ziegenberg <[email protected]>
moodlehq · Feb 7, 2025 · ef21453 · ef21453
1 parent c687fa9
commit ef21453
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/remote_branch_checker/remote_branch_checker.sh b/remote_branch_checker/remote_branch_checker.sh
@@ -316,6 +316,12 @@ mv ${WORKSPACE}/work/patchset.files.tmp ${WORKSPACE}/work/patchset.files
 if [[ ${versionbranch} -ge 405 ]]; then
     if grep -q 'UPGRADING.md\|upgrade.txt' ${WORKSPACE}/work/patchset.files; then
         echo "Error: The patchset contains changes to upgrade.txt or UPGRADING.md files." | tee -a ${errorfile}
+
+        dirtyupgrades="$( grep 'UPGRADING.md\|upgrade.txt' ${WORKSPACE}/work/patchset.files )"
+        if [[ -n "${dirtyupgrades}" ]]; then
+            echo "Error: File(s) affected:" | tee -a ${errorfile}
+            echo "${dirtyupgrades}" | sed 's/^/Error: /' | tee -a ${errorfile}
+        fi
     fi
 fi
 

diff --git a/tests/fixtures/remote_branch_checker/local_ci_fixture_upgrade_txt_for_405.regex b/tests/fixtures/remote_branch_checker/local_ci_fixture_upgrade_txt_for_405.regex
@@ -5,3 +5,8 @@
 condensedresult="smurf,error,1,0:overview,error,1,0
 <detail name="overview" status="error" numerrors="1" numwarnings="0"/>
 <message>The patchset contains changes to upgrade.txt or UPGRADING.md files.</message>
+<message>File(s) affected:</message>
+<message>UPGRADING.md</message>
+<message>lib/upgrade.txt</message>
+<message>mod/assign/UPGRADING.md</message>
+<message>mod/assign/upgrade.txt</message>