haskell/hie-bios

Turn unknown fields in `hie.yaml` into warnings

Aperta

#344 aperta il 13 apr 2022

Vedi su GitHub
 (1 commento) (1 reazione) (0 assegnatari)Haskell (70 fork)auto 404
ZuriHacgood first issue

Metriche repository

Star
 (198 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Until now, we threw an error message. The point being that you notice you didn't write the correct configuration and something is not right.

This will make it hard to introduce new optional keys, because older HLS versions will now choke on that config.

I think a warning is better. cabal-install also does this:

$ cabal build
Warning: /home/hasufell/git/stack2cabal/cabal.project: Unrecognized field
'foo' on line 11

Originally posted by @hasufell in https://github.com/haskell/hie-bios/issues/329#issuecomment-1061085723

Guida contributor

Direzione di ricerca
Esamina il codice di parsing della configurazione hie.yaml. Trova la parte che attualmente solleva un errore per campi sconosciuti e modificala per emettere un avviso. Aggiorna i test esistenti per riflettere il nuovo comportamento.
Tech stack
haskell
Dominio
toolingdeveloper experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
GitHaskell
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75