microsoft/Terminal

iTerm color schemes once more are failing to parse colors

Open

#352 aperta il 18 gen 2019

Vedi su GitHub
 (11 commenti) (3 reazioni) (0 assegnatari)C++ (3212 fork)batch import
Area-SettingsHelp WantedIssue-BugProduct-ColortoolWork-Item

Metriche repository

Star
 (35.764 star)
Metriche merge PR
 (Merge medio 27g 19h) (24 PR mergiate in 30 g)

Descrizione

Not sure if I am supposed to request to get this re-opened or if I should create another bug. But this is the same as in #2

  • Your Windows build number: (Type ver at a Windows Command Prompt) Microsoft Windows [Version 10.0.17763.253]

  • What you're doing and what's happening: (Copy & paste specific commands and their output, or include screen shots) Running: PS C:\Tools\ColorTool> .\ColorTool.exe C:\Tools\ColorTool/schemes/molokai.itermcolors Error loading ini file "C:\Tools\ColorTool/schemes/molokai.itermcolors" for key "DARK_BLACK" the value "" is invalid

The actual file contents can be found in this gist: https://gist.github.com/SneWs/4bbb50dce9a132a4dcbd07e3e2c85c44

  • What's wrong / what should be happening instead: The colors should be successfully parsed and set without any error messages being rendered.

Guida contributor

Direzione di ricerca
Ispeziona la logica di parsing INI in ColorTool (probabilmente nel codice sorgente) per capire perché una chiave con valore vuoto causa un errore. Traccia il parsing dei file itermcolors, in particolare la gestione dei valori dei componenti colore. Considera di correggere il parser per accettare stringhe vuote o saltare le chiavi non valide con garbo.
Tech stack
Nessuno
Dominio
clitooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++INI parsing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40