ER: Blank PRs get commented on for formatting issues too
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 67/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- github-actions, javascript
Direzione di ricerca
Leggi ~/github-actions/trigger-pr/check-linked-issue.js, in particolare la riga 105, dove l’issue segnala un errore quando pr.body è vuoto. Inizia esaminando come l’action gestisce i corpi delle pull request e consulta il rapporto sull’errore collegato in PR #8815; usa le guide fornite per testare GitHub Actions e provarla in un ambiente personale. Il lavoro è completato quando l’action gestisce un corpo PR vuoto e lascia il commento previsto.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Emergent Requirement - Problem
We should make sure that check-linked-issue.js can handle blank PRs as well. That code currently helps to check PRs for proper formatting to make sure an issue is linked, but it fails if the PR it is checking is blank.
Issue you discovered this emergent requirement in
- #8815
Date discovered
10-24-2026
Did you have to do something temporarily
- YES
- NO
Who was involved
@castillios
@daras-cu
What happens if this is not addressed
If a blank PR get published, the PR to check for a a properly linked issue does not leave any comment on the PR to notify the author of an issue.
Resources
- This comment outlines what that failure looks like and has links to the errors that come up: https://github.com/hackforla/website/pull/8815#discussion_r4367949963
- For testing Github Actions, refer to https://github.com/hackforla/website/issues/6537#issuecomment-2041147335, especially Tips 6, 7, and 8, if you have not already created you personal environment for testing.
- More notes on testing Gtihub Actions: How to Test GitHub Actions
- Additional Notes for GitHub Actions
Recommended Action Items
- Make a new issue
- Discuss with team
- Let a Team Lead know
Potential solutions [draft]
Issue:
- Complexity: Small (While the suggested change here only requires changing one line of code, I gave it the Complexity: Small label because it requires testing GitHub Actions.)
- Relationships: None
- Size: 0.25pt
- Fix:
in ~/github-actions/trigger-pr/check-linked-issue.js (line 105), change:
const match = pr.body.match(regex);
... to:
const match = (pr.body ?? '').match(regex);
- Lingua principale
- JavaScript
- Stelle
- 365
- Fork
- 872
- Merge medio
- 8g 19h
- PR unite (30g)
- 2
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di hackforla/website
-
Complexity: See issue making label epic good first issue Issue Making: Level 1 P-Feature: Project Info and Page Ready for Prioritization role: back end/devOps role: front end size: 0.25pt time sensitive
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
Complexity: Small ER Feature: Refactor GHA Issue Making: Level 2 ready for merge team role: back end/devOps size: 0.25pt
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
hackforla/website#8870 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Complexity: Small P-Feature: Project Info and Page ready for merge team role: back end/devOps role: front end size: 0.25pt time sensitive
Difficoltà 1/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno
-
Complexity: Small ER Feature: Refactor CSS Issue Making: Level 2 P-Feature: Projects page ready for merge team role: front end size: 0.25pt
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
hackforla/website#8860 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue P-Feature: Project Info and Page Ready for Prioritization role: back end/devOps role: front end size: 0.25pt
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di hackforla/website
Issue simili
-
Add: QudsunaTV.psApertachannels:add check:passed
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 66/100
iptv-org/database#39416 · 2 commenti ·
I maintainer di solito rispondono entro 2 giorni
-
[TEXT] Flache Karte ist 2DAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
swisstopo/topo-swisseobrowser#48 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
I maintainer di solito rispondono entro 1 giorno