facebookarchive/prepack

Adapt evaluators to deal with abstract conversion values

Open

#1109 aperta il 27 ott 2017

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (520 fork)batch import
abstracthelp wantedlevel 3 (medium)test needed

Metriche repository

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

Descrizione

The evaluators for binary and unary expression already have logic that allows them to deal with abstract operands. This does not cover the case where a concrete operand may become abstract after implicit conversion.

This pull request fixes this for the + operator: #1108. Follow that model for the other operators.

Guida contributor

Direzione di ricerca
Esamina la pull request #1108 che implementa la correzione per l'operatore '+' e applica lo stesso modello agli altri operatori negli evaluatori.
Tech stack
javascript
Dominio
tooling
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70