chore: Try to get rid of `tryConvertSnippet`'s special case for `UnknownData`
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 70/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- compilers
Direzione di ricerca
Start at the two tryConvertSnippet call sites shown in the issue and inspect the existing withDataType entry point. Compare behavior for non-integer types and UnknownData, then verify that both conversions follow the same rules without the special case.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Right now, tryConvertSnippet behaves differently based on how many arguments it got.
discriminantExpr = tryConvertSnippet(this.ctx, discriminantExpr, [i32, u32]);
const switchType = discriminantExpr.dataType;
invariant(switchType !== UnknownData);
caseExprs = caseExprs.map(([test, consequent]) => [
test === switchDefault ? test : tryConvertSnippet(this.ctx, test, [switchType]),
consequent,
]);
The first tryConvertSnippet throws when receiving non integer type. The second tryConvertSnippet does not, because there's an UnknownData edge case that let's it through.
Let's try to make the behavior consistent, we can use withDataType instead of tryConvertingSnippet.
- Lingua principale
- TypeScript
- Stelle
- 3.3k
- Fork
- 135
- Merge medio
- 4g 1h
- PR unite (30g)
- 65
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di software-mansion/TypeGPU
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
software-mansion/TypeGPU#3219 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
software-mansion/TypeGPU#3213 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
software-mansion/TypeGPU#3008 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
software-mansion/TypeGPU#2948 · 2 commenti ·
I maintainer di solito rispondono entro 2 giorni
-
docs: Fix camera orientation in "Monocular Light Injection" exampleForse già presa @pullfrog l’ha presa 60 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
software-mansion/TypeGPU#2890 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di software-mansion/TypeGPU
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
simkimsia/netlify-axi#19 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
subhranshudash13-dotcom/Contribo#74 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement frontend good first issue Stellar Wave trivial
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Godgift-care/sci-healthcare-frontend#166 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
backend good first issue security Stellar Wave trivial
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Godgift-care/sci-healthcare-backend#171 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno