Skip to content
This repository has been archived by the owner on Jul 10, 2024. It is now read-only.

Maven repair, create readme #1

Merged
merged 15 commits into from
Jul 24, 2019
Merged

Conversation

eogra7
Copy link
Contributor

@eogra7 eogra7 commented Jul 24, 2019

This PR includes changes to tidy up the repo by removing compiled files, adding a build script, and beginning documentation,

@eogra7 eogra7 closed this Jul 24, 2019
@eogra7 eogra7 reopened this Jul 24, 2019
@eogra7 eogra7 changed the title Develop Maven repair, create readme Jul 24, 2019
@@ -184,7 +184,7 @@ public void setRoom(Location l, String submissionId) {
spawnHologram(bl.clone().add(.5, .5, .5), title);
}

spawnHologram(bl.clone().add(.5, -.25, .5), "�6"+s.getScore());
spawnHologram(bl.clone().add(.5, -.25, .5), "" + (char)(0xfeff00a7) + "6" + s.getScore());
Copy link

Choose a reason for hiding this comment

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

Any reason why this isn't using ChatColor api?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Would you like to implement it? I created this issue #2 for you.

@darbyjack
Copy link
Contributor

darbyjack commented Jul 24, 2019

Hey there, super active developer in the MC community, once this PR is accepted, I'd be happy to go through and clean it up a tad more for you. Yes maven might be annoying if you're not used to it, but I'd be happy to take a few moments and clean it up!

@Delta2Force Delta2Force merged commit 6cc45e6 into Delta2Force:master Jul 24, 2019
@Delta2Force
Copy link
Owner

Delta2Force commented Jul 24, 2019

Wow! Thank you so much.

@Delta2Force
Copy link
Owner

Delta2Force commented Jul 24, 2019

Im kind of worried that no one changed the names of my „I hate maven“ and „IAHTRE“ repos hahaha

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

4 participants