srid/haskell-flake

Add `cabal check` to `checks`

Ouverte

#50 ouverte le 1 déc. 2022

Voir sur GitHub
 (0 commentaire) (0 réaction) (0 personne assignée)Nix (26 forks)github user discovery
enhancementgood first issue

Métriques du dépôt

Stars
 (231 étoiles)
Métriques de merge PR
 (Merge moyen 9h 49m) (1 PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Inspectez les vérifications existantes dans le module flake et implémentez une nouvelle vérification en utilisant Setup.hs ou cabal install.
Stack technique
haskell
Domaine
build systemdevops
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
NixHaskellcabal
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60