Skip to content

a new enemy named "WIzard" added to the game.#138

Open
hansalkothari wants to merge 1 commit intotagniam:masterfrom
hansalkothari:master
Open

a new enemy named "WIzard" added to the game.#138
hansalkothari wants to merge 1 commit intotagniam:masterfrom
hansalkothari:master

Conversation

@hansalkothari
Copy link
Copy Markdown

I added 3 new files to the project , to create a new enemy named "Wizard".

Copy link
Copy Markdown
Owner

@tagniam tagniam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just one small fix needed

Comment thread include/Enemy.h
etDemon,
etNumEnemyTypes
etNumEnemyTypes,
etWizard
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please put etWizard before etNumEnemyTypes, etNumEnemyTypes must be last with the way the game's enemy selector works :)

@tagniam
Copy link
Copy Markdown
Owner

tagniam commented Oct 30, 2020

@hansalkothari any update? Just making sure I'd have enough time to merge this to count for Hacktoberfest!

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