popcodeorg / popcodeorg/popcode
Don’t suggest enabling p5 based on global usage
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 191
- Forks
- 143
- Ø Merge
- 4 T. 11 Std.
- Gemergte PRs (30 T.)
- 5
Beschreibung
P5 defines quite a few global variables and functions, many of which have common words for names. Suggesting enabling P5 every time someone references an undefined variable like day is unlikely to be the most helpful suggestion.
Practical suggestion: only auto-suggest library globals if the library defines three or fewer globals in total.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit src/config/libraries.js, insbesondere mit den im Issue verknüpften globalen Definitionen von p5, und finde anschließend den Code, der Vorschläge für undefinierte Variablen generiert. Überprüfe das aktuelle Verhalten für einen häufigen Namen wie day; die Aufgabe ist erledigt, wenn Bibliotheken mit mehr als drei Globals nicht mehr automatisch vorgeschlagen werden, während kleinere Bibliotheken weiterhin infrage kommen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100