[Chapter 66] Missing `return` in `spell_trigger`
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 50/100
Línea de trabajo
Lee book/src/chapter_66.md alrededor de las líneas 453-465 e inspecciona el flujo de spell_trigger después de la comprobación de maná. Confirma que la ruta de maná insuficiente todavía puede lanzar el hechizo y, después, resuelve si esa ruta debería devolver return o si la comprobación pertenece a otro lugar; se considera terminado cuando una llamada directa no puede lanzar el hechizo sin maná suficiente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
There is a missing return statement after the check on the mana pool.
Currently, if the caster lacks the mana for a spell, the pool won't be updated but the spell will still be cast.
For the player, this won't happen because the spell will be cast from the targeting menu, which has the proper check against this, but a direct call to spell_trigger could lead to an error.
(If the check is always supposed to always have been done prior to the call, then let's simply remove it here.)
- Lenguaje dominante
- Rust
- Estrellas
- 967
- Forks
- 165
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de amethyst/rustrogueliketutorial
-
Wrong typing in chapter 57aAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
[RESOLVED] Wayland IssueAbierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 45/100
Todos los issues de amethyst/rustrogueliketutorial
Issues similares
-
The book has a Nix install section that chelis.ch lacks, and it names two of the three systemsAbiertoarea:ecosystem documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Chelis-Lang/chelis#3866 ·
Los mantenedores suelen responder en 1 día
-
bug documentation sev-0
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
dekobon/big-code-analysis#1620 ·
Los mantenedores suelen responder en 3 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100