AOSSIE-Org / AOSSIE-Org/Template-Repo

Fix: Hard-coded checklist items in dangerfile.js don't match PR template

Offen
#112 0 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @kpj2006 Auf GitHub ansehen
Vorherrschende Sprache
YAML
Sterne
17
Forks
29
Ø Merge
3 T. 13 Std.
Gemergte PRs (30 T.)
6

Beschreibung

## Summary

The `requiredChecklistItems` array in `dangerfile.js` contains hard-coded checklist strings that don't match the actual PR template checklist items. This causes compliant PRs to be incorrectly flagged by the Danger CI check.

## Problem

The current strings in `dangerfile.js` are:
- `"My PR addresses a single issue"`
- `"My code follows the project's code style"`
- `"My changes generate no new warnings or errors"`

However, the actual PR template checklist contains different text, causing false negatives.

## Proposed Fix

Update `requiredChecklistItems` to exactly match the PR template text, or implement tolerant/label-based matching (case-insensitive, whitespace-normalized) to avoid future drift.

## References

- PR: https://github.com/AOSSIE-Org/Template-Repo/pull/111
- Review comment: https://github.com/AOSSIE-Org/Template-Repo/pull/111#discussion_r3005886058

Reported by: @kpj2006

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.