pytest-dev / pytest-dev/pytest-html
Expand ESLint
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 779
- Forks
- 260
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
ESLint was introduced in https://github.com/pytest-dev/pytest-html/pull/367
This first pass included almost all relevant ESLint sections except for the following:
The following three rules were also skipped, since they involved making broad changes across the JS code. It would be better to submit them as separate PRs:
- https://eslint.org/docs/rules/no-use-before-define
- https://eslint.org/docs/rules/#strict-mode
https://eslint.org/docs/rules/camelcase - PR open in https://github.com/pytest-dev/pytest-html/pull/410- Done
I'll be adding all of the above, one PR per bullet point when I find the time to.
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 der Durchsicht der in Pull Request 367 eingeführten ESLint-Konfiguration sowie der im Issue verlinkten Regelabschnitte Stylistic Issues und ECMAScript 6. Behandle die verbleibenden Regelgruppen als separate Änderungen und betrachte die Arbeit als abgeschlossen, wenn die ausgewählten Regeln konfiguriert sind und das JavaScript-Linting ohne nicht zusammenhängende umfangreiche Änderungen erfolgreich durchläuft.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- eslint, javascript
- Bereich
- tooling
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 25/100