srid/haskell-flake

Add `cabal check` to `checks`

Aperta

#50 aperta il 1 dic 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Nix (26 fork)github user discovery
enhancementgood first issue

Metriche repository

Star
 (231 stelle)
Metriche merge PR
 (Merge medio 9h 49m) (1 PR mergiata in 30 g)

Descrizione

In arion I had a separate build with

                  checked = super.haskell.lib.overrideCabal pkg (o: {
                    postConfigure = ''${o.postConfigure or ""}
                      if ! ${hsuper.cabal-install}/bin/cabal check;
                      then
                        echo 1>&2 ERROR: cabal file is invalid. Above warnings were errors.
                        exit 1
                      fi
                    '';
                  });

I think we could touch the outputs and exit 0 to speed this up. Doing it in runCommand would be fragile. This should probably call Setup.hs instead of cabal-install.

Running it as part of the main build isn't ideal because the check may be more strict than necessary. I think it should only run in CI.

Guida contributor

Direzione di ricerca
Ispeziona i controlli esistenti nel modulo flake e implementa un nuovo controllo usando Setup.hs o cabal install.
Tech stack
haskell
Dominio
build systemdevops
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
NixHaskellcabal
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60