Skip to content

Conversation

@actioninja
Copy link

Also fixed quite a few bugs.

@Iamgoofball
Copy link

this PR has seen some shit over the years, can we get a merge bros?

@werty1124
Copy link
Owner

This is the first time I have seen this PR. It only took me 5 years sorry!

@Kylerace
Copy link

Kylerace commented Apr 2, 2021

@werty1124 dont worry, unmerged prs age like fine wine. at least thats what maintainers keep telling me

@bobbah
Copy link

bobbah commented Apr 2, 2021

we did it

@Kylerace
Copy link

Kylerace commented Apr 2, 2021

we did it

its not merged yet unfortunately it has to be reviewed

Copy link

@Kylerace Kylerace left a comment

Choose a reason for hiding this comment

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

minor grammar errors

else
{
playerIn.addChatMessage(new ChatComponentText("you can feel a strange presence"));
playerIn.addChatMessage(new TextComponentString("you can feel a strange presence"));
Copy link

Choose a reason for hiding this comment

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

Suggested change
playerIn.addChatMessage(new TextComponentString("you can feel a strange presence"));
playerIn.addChatMessage(new TextComponentString("You can feel a strange presence."));

if(!worldIn.isRemote)
{
playerIn.addChatMessage(new ChatComponentText("You feel weak after transfering energy to the crystal"));
playerIn.addChatMessage(new TextComponentString("You feel weak after transfering energy to the crystal"));
Copy link

Choose a reason for hiding this comment

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

Suggested change
playerIn.addChatMessage(new TextComponentString("You feel weak after transfering energy to the crystal"));
playerIn.addChatMessage(new TextComponentString("You feel weak after transferring energy to the crystal."));

@Kylerace
Copy link

Kylerace commented Apr 2, 2021

my work here is done 😎

@stylemistake
Copy link

Still not merged smh 😔

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.

6 participants