Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create Consumable Object #63

Merged
merged 3 commits into from
Jun 21, 2021
Merged

Create Consumable Object #63

merged 3 commits into from
Jun 21, 2021

Conversation

Mitchell-Borders
Copy link
Collaborator

No description provided.

…n the upgrade class where there was no () for the super from_json.
Copy link
Collaborator

@erickbickler erickbickler left a comment

Choose a reason for hiding this comment

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

Just one minor change

@erickbickler erickbickler linked an issue Jun 11, 2021 that may be closed by this pull request
@erickbickler
Copy link
Collaborator

erickbickler commented Jun 11, 2021

Please also update test_shooter unit tests to match new consumable object init, make sure the tests run.

…spawnable boolean. When I tried running tests they wouldn't run unless I had a something set up, so thats what the .vscode/settings.json is.
@Mitchell-Borders Mitchell-Borders changed the title Made the consumable class, and consumable enum. Create Consumable Object Jun 12, 2021
…ass. There were two wall classes, so I deleted the older one and kept the newer one. Finally, I added tests for initialization of the upgrade and consumable object.
Copy link
Collaborator

@erickbickler erickbickler 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 to me

Copy link
Owner

@HagenSR HagenSR left a comment

Choose a reason for hiding this comment

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

GJ!

@HagenSR HagenSR merged commit 129f32b into dev Jun 21, 2021
@HagenSR HagenSR deleted the Create-Consumable-Object branch June 21, 2021 19:51
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.

Create Consumable Object
3 participants