-
Notifications
You must be signed in to change notification settings - Fork 280
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
Project maven cleanup #50
base: master
Are you sure you want to change the base?
Conversation
TODO: cleanup NMS modules
What are the changes for NMS compat modules aiming for? Did you have trouble building? |
It was not following the maven standards, i also provided a repository containing NMS, so it will be possible to build it without having to run buildtools first. |
I currently have no space for reviewing which parts are really "necessary"... Specifically:
|
|
"Improve something" means i have to look through every line - especially if it's not at all crucial, it's probably no option at this very moment :). There is too many type of changes mixed in the commits and this PR. With MC 1.13 in the queue there is no way to look further into it. Best would be to separate:
This'll have to wait until i'm through with most of 1.13. |
Ok, i'll wait the 1.13 release first and then start splitting my changes into more PRs ;) |
Thank you :). |
who wants to get taht crpto hacl |
TODO:
I also added NCP to my CI setup, i added admin rights over the folder to any member of the NCP github org (authentication works via github oauth, remember allow access to the org on first login).
Jenkins build: https://ci.codemc.org/job/NoCheatPlus/job/NoCheatPlus/
Maven repository: https://repo.codemc.org/repository/maven-public/