firefox-devtools/profiler

Fix the types of some of the ProfilerOverheadStats object fields

Aperta

#2856 aperta il 8 ott 2020

Vedi su GitHubGuida contributor
 (23 commenti) (0 reazioni) (0 assegnatari)TypeScript (486 fork)auto 404
code maintenancehelp wantedready

Metriche repository

Star
 (1515 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Studia le definizioni di tipo esistenti in src/types/profile.js, comprendi i tipi Microseconds e Percentage, quindi aggiorna di conseguenza il tipo OverheadStats. Esegui yarn flow e yarn lint js per verificare.
Tech stack
typescript
Dominio
performance
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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.
65