firefox-devtools/profiler

Fix the types of some of the ProfilerOverheadStats object fields

Offen

#2.856 geöffnet am 08.10.2020

Auf GitHub ansehenContributor Guide
 (23 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (486 Forks)auto 404
code maintenancehelp wantedready

Repository-Metriken

Stars
 (1.515 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

We have a type for Overhead statistics, and all its values are Microseconds right now: https://github.com/firefox-devtools/profiler/blob/dcb8c42af81c7ee6f14c1fa5be9881dc5d03c115/src/types/profile.js#L459-L482

But some of it's values are not actually Microseconds. For example samplingCount is just a number and overheadPercentage is a percentage. So we should fix these types to reflect the actual values. For percentage, since we don't have a Percentage type, we need to create that type under the InnerWindowID and will look pretty similar since it's also going to use the number esentially: https://github.com/firefox-devtools/profiler/blob/dcb8c42af81c7ee6f14c1fa5be9881dc5d03c115/src/types/profile.js#L31

After making these changes, you should make sure flow and linter is happy by running yarn flow and yarn lint-js.

┆Issue is synchronized with this Jira Task

Contributor Guide

Research-Richtung
Studieren Sie die vorhandenen Typdefinitionen in src/types/profile.js, verstehen Sie die Typen Microseconds und Percentage und aktualisieren Sie dann entsprechend den OverheadStats Typ. Führen Sie yarn flow und yarn lint js zur Überprüfung aus.
Tech Stack
typescript
Domain
performance
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Gitjavascript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65