Repository navigation
fix: correções da revisão do PR #31 - #32
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Correções dos achados da revisão do PR #31 (dev → main). Cada correção está num commit próprio e tem um teste de regressão, que falha sem a correção.
Correções
a0faf4e).rearmWeaponagora devolve o gatilho, a mira de interruptor e a recarga como presos, como já fazemrespawnCharactererearmMelee. Assim, segurar o botão durante a morte, o respawn, o reinício ou a revanche não dispara nem abre a mira: é preciso apertar de novo. A mira de segurar fica de fora porque abre por estar presa, não pela borda. Os bots também passam pelorearmWeapone só esperam um tick a mais depois de renascer. Quatro testes da sessão apertavam o botão já no primeiro quadro e agora soltam um quadro antes.4ad10d1refactor +fff8df0fix). A troca de laço saiu do adapter paraaudio/loopSwitch.ts, que é testado sem navegador. Cada troca recebe um número (latestRequest), e só a pedida por último começa uma fonte. Antes, as sequências A→B→A e A→nada→A deixavam duas fontes tocando.adabd76).holdStillsolta a corda. Na morte, o aviso de alcance apaga. EmendMatch, o corpo para e o aviso de alcance também apaga.df4945e).awardMedalserecordKillagora acrescentam ao vetor em vez de limpá-lo, e cada kill pontua só as próprias medalhas (medalPointsFrom, sem alocar). A sessão limpamedalsebotMedalsno começo do quadro.395ec57refactor +aaf71c1fix). O FNV-1a foi para o núcleo (math/fnv1a.ts) e olayoutFingerprintpassou a usá-lo.botBallotsagora semeia cada bot pelo hash do id, então mandar um bot do meio para o banco não muda o voto dos outros.1dcc1af). O golpe agora usabot.melee.weapon.87992e8). Quem volta do banco já ganha spawn noseatBotse não é posto de novo emseatBotsOnTheBoard. O teste conta as tiradas da semente.638cb6f, refactor).grappleEyeMsaiu, eeyeOffoi paramovement/characterState.ts, ao lado deEYE_HEIGHT_RATIO. O gancho e os combatentes usam a mesma função, ematch/continua dependendo demovement/, nunca o contrário.Verificação
npm run verifypassa: lint, tipos e 3853 testes com 100% de cobertura.