_getMessage should be inlined and/or renamed
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 35/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- mobile-dev
Direzione di ricerca
Inizia in index.js intorno alla riga 126, dove _getMessage viene usato per preparare i dati per this._iosDeviceLibStdioHandler.writeData. Conferma che questo sia il suo unico utilizzo, quindi decidi se incorporare la serializzazione direttamente o rinominare l’helper in modo che descriva la chiamata al metodo nativo. Il lavoro è completato quando l’unico utilizzo rimane chiaro e l’helper obsoleto o rinominato è risolto.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
I think _getMessage should be:
- Inlined as we are using it on only one place. I would assign the one line to a variable and pass it to this._iosDeviceLibStdioHandler.writeData
- Named something like _serializeNativeMethodCall... something that will ring the right bell when you read it.
https://github.com/telerik/ios-device-lib/blob/382c9c63dd1999ef247def16a3787b5987d63649/index.js#L126
- Lingua principale
- C++
- Stelle
- 16
- Fork
- 8
- Merge medio
- 19h 29m
- PR unite (30g)
- 2
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di NativeScript/ios-device-lib
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
Sometimes this is showing only 1 device while idevice_id from libimbile shows all connected devices Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
NativeScript/ios-device-lib#73 · 1 commento ·
-
Error on communication Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 20/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
NativeScript/ios-device-lib#66 · 2 reazioni ·
Tutte le issue di NativeScript/ios-device-lib
Issue simili
-
enhancement
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
QuantStack/git2cpp#187 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
mlcommons/mobile_app_open#1182 ·
-
Needs-Triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
microsoft/winget-cli#6547 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
AXERA-TECH/ax-llm#77 ·