-
-
Notifications
You must be signed in to change notification settings - Fork 318
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
Add a deno pet #567
base: master
Are you sure you want to change the base?
Add a deno pet #567
Conversation
Erm. This is amazing. Well done on the PR. I'll take some time to try this out with a few different scenarios then give a +1 |
@@ -178,6 +179,8 @@ export function createPet( | |||
return new Cat(...standardPetArguments, PetSpeed.normal); | |||
case PetType.chicken: | |||
return new Chicken(...standardPetArguments, PetSpeed.normal); | |||
case PetType.deno: | |||
return new Deno(...standardPetArguments, PetSpeed.normal); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've been playing around with a couple of these and think the pet animations might look more natural at slow
instead of normal
. It would walk and run slower (fewer pixels/sec), but it is a big ol' dinosaur.
What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awww yeah, could be kind of cute lumbering around
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #567 +/- ##
==========================================
+ Coverage 16.80% 17.07% +0.26%
==========================================
Files 22 23 +1
Lines 1166 1177 +11
Branches 158 158
==========================================
+ Hits 196 201 +5
- Misses 964 970 +6
Partials 6 6 ☔ View full report in Codecov by Sentry. |
Added a lil dinosaur pet to represent Deno.