GoogleChrome/lighthouse-ci

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

Open

#375 aperta il 8 lug 2020

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)JavaScript (658 fork)batch import
P3enhancementgood first issue

Metriche repository

Star
 (5965 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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?

Guida contributor

Direzione di ricerca
La richiesta del problema è di cambiare la conclusione del controllo GitHub da 'success' a 'neutral' quando tutti gli assert falliti sono avvisi. Indaga la logica di asserzione nel codice di lighthouse ci, probabilmente nel modulo 'assert'. Esamina come viene creato l'esecuzione del controllo e dove viene impostata la conclusione. Considera l'uso delle GitHub Checks API per aggiornare la conclusione a 'neutral' mantenendo comunque il controllo complessivo come superato. Fai riferimento all'implementazione esistente e alla documentazione delle GitHub API per le esecuzioni dei controlli.
Tech stack
javascriptnodejs
Dominio
backendci cd
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75