[FIXUP]

Author: HannesWell

.github/workflows/checkAcknowledgements.yml

@@ -12,10 +12,12 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+      with:
+        fetch-depth: 0
     - name: Determine changed files
       id: changed-files
       run: |
-        changedFiles=$(git diff --name-only --diff-filter=ACMR ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }} | grep acknowledgements.md$)
+        changedFiles=$(git diff --name-only --diff-filter=ACMR ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }} | grep acknowledgements.md$ | xargs)
         echo "Changed files: ${changedFiles}"
         echo "files=${changedFiles}" >> "$GITHUB_OUTPUT"
     - name: Check modified acknowledgements files
@@ -24,27 +26,22 @@ jobs:
       with:
         script: |
           const fs = require('fs')
-          const files = '${{ steps.changed-files.outputs.files }}'.trim().split(/\r?\n/)
-          const nameIdRegex = /\[(?<name>[^\]]+)\]\(https:\/\/github\.com\/(?<id>[^\)]+)\)/g
-          const contributorNames = new Map()
+          const files = '${{ steps.changed-files.outputs.files }}'.trim().split(/\s+/)
+          const nameIdRegex = /\[(?<name>[^\]]+)\]\(https:\/\/github\.com\/(?<id>[^\)/]+)\)/g
           for (file of files) {
             let lines = fs.readFileSync(file, {encoding: 'utf8'}).split(/\r?\n/)
+            const contributorNames = new Map()
             for (line of lines) {
               for (match of line.matchAll(nameIdRegex)) {
                 computeIfAbsent(contributorNames, match.groups.id, () => new Set()).add(match.groups.name)
               }
             }
-          }
-          let boolean hasMultipleNames = false
-          for (const [profile, names] of contributorNames) {
-            if (names.size > 1) {
-              core.error('Multiple names found for profile '" + profile + "': ' + Array.from(names).join(', '))
-              hasMultipleNames = true
+            for (const [profile, names] of contributorNames) {
+              if (names.size > 1) {
+                core.setFailed("Multiple names found for profile '" + profile + "': " + Array.from(names).join(', '))
+              }
             }
           }
-          if (hasMultipleNames) {
-            throw new Error('No selected name for profile: ' + profile)
-          }
           
           function computeIfAbsent(map, key, valueSupplier) {
             let value = map.get(key)

.github/workflows/generateAcknowledgements.yml

@@ -127,7 +127,7 @@ jobs:
                     profile = committerProfile
                   }
                   contributors.add(profile)
-                  computeIfAbsent(contributorNames, profile, () => new Set()).add(authorName)
+                  computeIfAbsent(contributorNames, profile, () => new Set()).add(authorName.trim())
                 } else { // author is null for directly pushed commits, which happens e.g. for I-build submodule updates
                   console.log("Skip commit of " + authorName)
                 }
@@ -144,11 +144,11 @@ jobs:
           const acknowledgementsFile = 'website/news/${{ inputs.eclipse-version }}/acknowledgements.md'
           let lines = fs.readFileSync(acknowledgementsFile, {encoding: 'utf8'}).split(/\r?\n/)
           
-          const nameIdRegex = /\[(?<name>[^\]]+)\]\(https:\/\/github\.com\/(?<id>[^\)]+)\)/g
+          const nameIdRegex = /\[(?<name>[^\]]+)\]\(https:\/\/github\.com\/(?<id>[^\)/]+)\)/g
           const existingContributorNames = new Map()
           for (line of lines) {
             for (match of line.matchAll(nameIdRegex)) {
-              existingContributorNames.set(match.groups.id, match.groups.name)
+              existingContributorNames.set(match.groups.id.trim(), match.groups.name.trim())
             }
           }
           // ------------------------------------------------------