Skip to content
This repository has been archived by the owner on Aug 18, 2020. It is now read-only.

Add damage indicators #69

Merged
merged 11 commits into from
Aug 4, 2019
Merged

Conversation

finnbainbridge
Copy link
Contributor

Added damage indicators from issue #68

@ForLoveOfCats
Copy link
Owner

ForLoveOfCats commented Jul 30, 2019

Can you squash the two commits before I do any review? That way the commit log is clean for the commits which modify anything which needs to change.

Edit for future clarity: The second commit redid all the work in the first so better to squash into one commit.

Damage Indicators now work perfectly
Copy link
Owner

@ForLoveOfCats ForLoveOfCats left a comment

Choose a reason for hiding this comment

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

Make sure to convert DamageIndicator.cs to use tab indentation

@ForLoveOfCats
Copy link
Owner

and of course if I said anything inaccurate please do correct me

Copy link
Owner

@ForLoveOfCats ForLoveOfCats left a comment

Choose a reason for hiding this comment

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

You made the image yourself right?

@finnbainbridge
Copy link
Contributor Author

Yes, I did make the image myself

@ForLoveOfCats ForLoveOfCats merged commit 79d3333 into ForLoveOfCats:master Aug 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants