apache / apache/cordova-node-xcode

Cleanup

Offen
#78 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement help wanted
Vorherrschende Sprache
JavaScript
Sterne
217
Forks
102
Ø Merge
2 T. 3 Std.
Gemergte PRs (30 T.)
6

Beschreibung

From review of PR #56 (watch2 support), I have a feeling that we should be able to clean up both the source code and all of the tests. I am raising this for future consideration and discussion.

Starting to add a list:

- [ ] use const & let instead of var, especially in new code (noticed after merging PR #56)
- [ ] look for ways to make the code easier to read & understand, especially the code added in PRs #56 & #76

/cc @l3ender

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Überprüfe zuerst PR #56 und PR #76 und untersuche anschließend den Quellcode und die Tests, die von ihren Änderungen betroffen sind. Teile die allgemeine Aufräumanfrage in konkrete Aufgaben zur Verbesserung der Lesbarkeit und für var-to-const/let auf, wobei jede Aufgabe ihren Code und ihre Tests benennt und den erwarteten abgeschlossenen Zustand definiert.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, node.js
Bereich
tooling
Issue-Typ
Refactoring
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
20/100

Neue Issues direkt in Ihr Postfach

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