GoogleChrome/lighthouse

runWarnings format difficult for pragmatic usage

Open

#11 352 ouverte le 1 sept. 2020

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)JavaScript (9 278 forks)batch import
P3breakinggood first issue

Métriques du dépôt

Stars
 (27 768 stars)
Métriques de merge PR
 (Merge moyen 18j 18h) (40 PRs mergées en 30 j)

Description

I have two concerns with runWarnings specifically around pragmatic usage:

  • runWarnings contains both errors (eg, page hung / non-successful request on main document resource) and warnings (eg, CPU throttling might be bad)
  • The errors and warnings contained within runWarnings are strings. There's no handle to be able to identify one warning/error from another, without searching the string.

Feature request summary

runWarnings to be changed to an object. It should contain code and message. Just like runtimeError.

What is the motivation or use case for changing this?

Node.js/API/PageSpeed users of Lighthouse will be able to handle errors/warnings reliably.

Guide contributeur

Direction de recherche
Identifiez l'endroit où runWarnings est généré dans le code de Lighthouse, généralement dans l'audit ou le générateur de rapports. Utilisez runtimeError comme modèle et convertissez runWarnings d'un tableau de chaînes en un tableau d'objets avec les propriétés 'code' et 'message'.
Stack technique
javascriptnodejs
Domaine
apiperformance
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.
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