GoogleChrome/lighthouse-ci

Report GitHub check status differently when highest failing assert is a warning

Open

#375 ouverte le 8 juil. 2020

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)JavaScript (658 forks)batch import
P3enhancementgood first issue

Métriques du dépôt

Stars
 (5 965 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

I've currently set a assert profile like this:

module.exports = {
  ci: {
    ...
    assert: {
      ...
      assertions: {
        "categories:performance": "warn",
        "categories:best-practices": "warn",
        "categories:seo": "warn",
        "uses-long-cache-ttl": "off",
        "uses-http2": "off"
      }
    }
  }
}

What I get on github is this:

The message is fine, but having a success conclusion makes the checks panel to auto-collapse, so users won't be attracted to it at all. Maybe we should use neutral or find another way to improve that, but without failing the check?

Guide contributeur

Direction de recherche
Cette demande propose de changer la conclusion du contrôle GitHub de 'success' à 'neutral' lorsque toutes les assertions échouées sont des avertissements. Étudiez la logique d'assertion dans le codebase de lighthouse ci, probablement dans le module 'assert'. Examinez comment l'exécution du contrôle est créée et où la conclusion est définie. Envisagez d'utiliser l'API GitHub Checks pour mettre à jour la conclusion à 'neutral' tout en conservant le contrôle global comme réussi. Référez vous à l'implémentation existante et à la documentation de l'API GitHub pour les exécutions de contrôle.
Stack technique
javascriptnodejs
Domaine
backendci cd
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75