github / github/accessibility-alt-text-bot
🐛 Bug: No fallback behavior for action runs on forks
- Vorherrschende Sprache
- JavaScript
- Sterne
- 101
- Forks
- 20
- Ø Merge
- 21 Std. 53 Min.
- Gemergte PRs (30 T.)
- 5
Beschreibung
When this action runs on a fork and doesn't have permissions to fetch comments _(am I using the right terminology?)_, we see the dreaded `Resource not accessible by integration` error. Example from https://github.com/JoshuaKGoldberg/create-typescript-app/pull/1110 -> https://github.com/JoshuaKGoldberg/create-typescript-app/actions/runs/7297169422/job/19886082551?pr=1110:
```plaintext
Run github/accessibility-alt-text-bot@v1.4.0
Run source /home/runner/work/_actions/github/accessibility-alt-text-bot/v1.4.0/queries.sh
Detected bad alt text: true
Event type: pr_description
GraphQL: Resource not accessible by integration (addComment)
Error: Process completed with exit code 1.
```
This is a similar issue to https://github.com/mtfoley/pr-compliance-action/issues/368 -> https://github.com/mtfoley/pr-compliance-action/pull/374.
Suggestion: can the bot log an informative message in the error about what it _would_ post if it could? That way, if it's not able to actually post a comment, a useful error can still be logged.
Beitragsleitfaden
Rechercherichtung
Beginne damit, die im Issue gezeigte Ausführung der Action für einen Fork-Pull-Request zu reproduzieren, und verfolge den Pfad, der den Fehler `addComment` erzeugt. Als erledigt gilt die Aufgabe, wenn die Action nicht verfügbare Kommentarberechtigungen ohne einen undurchsichtigen Fehler behandelt und den Kommentarinhalt oder den im Issue beschriebenen anderen informativen Fallback protokolliert.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- github-actions, javascript
- Bereich
- ci-cd, devops
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100