NativeScript / NativeScript/nativescript-cli
Use constants instead of hardcoded `android` and `ios` strings
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 1.1k
- Forks
- 204
- Ø Merge
- 1 T. 9 Std.
- Gemergte PRs (30 T.)
- 8
Beschreibung
Is your feature request related to a problem? Please describe.
We have "android", "Android", "ANDROID" and "ios", "iOS", "IOS" hardcoded in many places in the CLI. In most of the places, we are also using platform.toLower() in order to workaround this.
Describe the solution you'd like
We should just use the same constants everywhere (e.g. DevicePlatformsConstants)
Beitragsleitfaden
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 in der NativeScript CLI und suche nach den hartcodierten Varianten von android, Android, ANDROID, ios, iOS und IOS sowie nach der bestehenden Verwendung von DevicePlatformsConstants. Ersetze die verstreuten Plattform-Strings durch die gemeinsamen Konstanten und überprüfe, dass alle relevanten CLI-Pfade dieselben Werte verwenden, ohne platform.toLower() zu benötigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- android, ios, javascript
- Bereich
- cli, mobile-dev
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100