Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Residence v5 #467

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

Residence v5 #467

wants to merge 23 commits into from

Conversation

PintaPepo
Copy link

Migrated project from Ant to Maven
Refactored NMS classes
Removed legacy economy/permissions integrations
Removed abandoned plugins
Hard dependency on Vault, we are in 2020.
Cleanup and refactored a lot of code
Added CI using Travis
Added code quality analysis using SonarCloud

And a lot more...

This was referenced May 9, 2020
Open
Closed
@PintaPepo
Copy link
Author

PintaPepo commented May 9, 2020 •
edited
Loading

Direct JAR download link
Outdated, get latest version using 'mvn package' inside the project folder after cloning the repo.

PintaPepo added 13 commits May 9, 2020 20:53
@PintaPepo
Update issue templates
c361d02
@PintaPepo
Update issue templates
da5f032
@PintaPepo
Consider Animals as: Animals or WaterMobs or Merchants. UTF-8 ordered…
85918b8 
… plugin.yml
@PintaPepo
Merge branch 'master' of https://github.com/PintaPepo/Residence
672a39c
@PintaPepo
Multiple code improvements
ae27836
@PintaPepo
Performance improvement: Use singleton instead of asList when there i…
4bd17f1 
…s only one element
@PintaPepo
Performance Improvements: String concatenation in loops, useless data…
b87ce16 
… structure initialization, etc
@PintaPepo
Delete useless StringBuilders
66bc662
@PintaPepo
Fix Bug: ServerLand calculation was comparing strings with == instead…
5c02945 
… of equals
@PintaPepo
More performance and code improvements
085cc53
@PintaPepo
Performance: Replace syncronizedMap with ConcurrentHashMap, remove co…
52ef510 
…nstructor
@PintaPepo
Fix: damageableProjectile, see Zrips#448 in parent repo
15c651b
@PintaPepo
Downgrade to Java 8, set Vault as a soft dependency
09c5742
@PintaPepo
Copy link
Author

PintaPepo commented May 10, 2020 •
edited
Loading

Summary of changes:

  • Migration from Ant to Maven
  • A lot of performance improvements
  • Code cleanup/reorganization
  • Added CI, and code analysis using Travis and SonarCloud: https://sonarcloud.io/dashboard?id=PintaPepo_Residence
  • Removed all NMS classes, could be added if we find a way to reference them without legal problems.
  • Added Issue Templates for GitHub issues
  • Bug Fix: Spectral arrows ignored animalkilling flag
  • Bug Fix: Cover all water mobs and all villager like mobs under the animal killing flag
  • Bug Fix: ServerLand comparison was using == instead of equals, which returned a wrong result

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@PintaPepo