compiler-explorer/compiler-explorer

Try and reduce deduplication in asm-parser-ewavr

Chiusa

#4689 aperta il 4 feb 2023

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (0 assegnatari)TypeScript (1605 fork)batch import
good first issuehelp wantedrequest

Metriche repository

Star
 (14.391 stelle)
Metriche merge PR
 (Merge medio 8g 5h) (62 PR mergiate in 30 g)

Descrizione

Is your feature request related to a problem? Please describe

It's a copy paste of vc parser with some (admittedly subtle) changes. Would be great to go over and see if there's anything beneficial to deduplicate.

Describe the solution you'd like

as above

Describe alternatives you've considered

Leaving it alone, and removing the TODO

Additional context

Not applicable

Guida contributor

Direzione di ricerca
Confronta asm parser ewavr con il parser vc, identifica la logica duplicata e proponi un refactoring per condividere il codice comune.
Tech stack
typescript
Dominio
backend
Tipo issue
Refactoring
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.
Abbastanza chiara
Prerequisiti
TypeScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50