Skip to content
This repository has been archived by the owner on Jun 29, 2023. It is now read-only.

More Pets :) #572

Merged
merged 46 commits into from
Jan 24, 2022
Merged

More Pets :) #572

merged 46 commits into from
Jan 24, 2022

Conversation

Chris6ix
Copy link
Contributor

What is the purpose of this pull request?

Adding more pets :)

How do your changes address the purpose?

They add new pets?

Changes

  • Add Vex Pet (Temporarily disabled)
  • Add Panda Pet
  • Add Fox Pet
  • Add Axolotl Pet (Temporarily disabled)
  • Add Piglin Pet (Temporarily disabled)
  • Add Slime Pet (Temporarily disabled) (Thanks to datatags)
  • Add Bee Pet
  • Add Goat Pet
  • Add Silverfish Pet
  • Small fixes in comments in MessageManager and COMPILING.md

Chris6ix and others added 30 commits January 13, 2022 18:57
Added it for v1_12_R1 as the Vex comes from MC 1.11
Adding it as v1_14_R1 as the Panda comes from MC 1.14
Just don't know if it should be under the "if (serverVersion.isAtLeast(ServerVersion.v1_14_R1))" or not...
Adding it as v1_14_R1 as the Fox comes from MC 1.14
Also moved the Panda Pet under a new "if (serverVersion.isAtLeast(ServerVersion.v1_14_R1))"
Adding it as v1_17_R1 as the Axolotl comes from MC 1.17
Also fixed a mistake in Fox pet that comes from copy/paste and a lack of concentration :)
Adding it as v1_16_R3 as the Piglin comes from MC 1.16
Adding it as v1_15_R1 as the Bee comes from MC 1.15
@datatags
Copy link
Collaborator

Looks great, thanks so much!

@datatags datatags merged commit b9f351c into iSach:master Jan 24, 2022
@Chris6ix
Copy link
Contributor Author

No problem, thanks again for your help :)

@Chris6ix Chris6ix deleted the Chris6ix-more-pets branch January 25, 2022 15:59
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.

None yet

2 participants