Skip to content

[lab-data-vikings] Fernando Costa#109

Open
breogann wants to merge 7 commits intoironhack-datalabs:masterfrom
datamad-content:lab-data-vikings
Open

[lab-data-vikings] Fernando Costa#109
breogann wants to merge 7 commits intoironhack-datalabs:masterfrom
datamad-content:lab-data-vikings

Conversation

@breogann
Copy link
Copy Markdown

No description provided.

@blancalluch
Copy link
Copy Markdown

Muy buen trabajo Fernando!
Te comento unas pequeñas cosas para mejorar.
El método attack no hace falta que lo vuelvas a definir dentro de las clases vikings y saxon porque estas clases ya heredan de soldiers que ya tiene ese método.
Una última cosa es que no puedes llamar a las variables con el mismo nombre que a las clases. Con poner Vikings en minúscula sirve como nombre de variable y Vikings con mayúscula para la clase.
Todo lo demás muy bien, enhorabuena!

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants