apache / apache/cordova-node-xcode
Cleanup
- 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
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