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.

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

feat(world & server): Implement Spawn Limit API #909

Merged
merged 6 commits into from
Feb 7, 2024
Merged

Conversation

thelooter
Copy link
Contributor

Description

Implements the Spawn Limits API

Checklist

The following items should be checked before the pull request can be merged.

  • Code follows existing style.
  • Unit tests added (if applicable).

@thelooter thelooter self-assigned this Nov 22, 2023
@thelooter thelooter requested a review from a team as a code owner November 22, 2023 21:46
Copy link

sonarcloud bot commented Nov 28, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
4.7% 4.7% Duplication

// TODO Auto-generated method stub
throw new UnimplementedOperationException();
Preconditions.checkArgument(spawnCategory != null, "SpawnCategory cannot be null");
Preconditions.checkArgument(spawnCategory != SpawnCategory.MISC,
Copy link
Contributor

Choose a reason for hiding this comment

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

This check does not consider if a new spawn limit has been implemented in paper. It would be good if something like an UnimplementedOperationException gets thrown whenever that is the case.

@thelooter thelooter enabled auto-merge (squash) February 7, 2024 17:33
Copy link

sonarcloud bot commented Feb 7, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
4.7% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

@Thorinwasher Thorinwasher left a comment

Choose a reason for hiding this comment

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

Looks good to me

@thelooter thelooter merged commit ca614fc into v1.20 Feb 7, 2024
3 checks passed
@thelooter thelooter deleted the feat/spawn-limits branch February 7, 2024 17:35
thelooter added a commit that referenced this pull request Feb 13, 2024
* fix playerlist hasplayedbefore (#944)

* Implemented more methods in SkullMetaMock and PotionEffectType (#945)

* Implemented more functions in SkullMetaMock

* Implemented createEffect in PotionEffectType

* fix merge conflicts

* Requested changes

* feat(entities): Implement Rest of Chicken Methods (#925)

Co-authored-by: Thorinwasher <37477983+Thorinwasher@users.noreply.github.com>

* chore(stubbing): Add method stubs for visual Blockdata API in TNTPrimed (#950)

* chore(stubbing): Stub Paper Methods (#951)

* Implement getOfflinePlayerIfCached with tests (#947)

* Implement getOfflinePlayerIfCached with tests

* Fix formatting

* Fix more formatting

* Adds Mocks for both Boat and ChestBoat (#851)

* Adds Mocks for both Boat and ChestBoat

* chore(boats): Implement Changes

* feat(entities): Remove Loottable API from Chestboat and add getType

* Fix formatting

---------

Co-authored-by: Thorinwasher <ragnarlol96@gmail.com>
Co-authored-by: Thorinwasher <37477983+Thorinwasher@users.noreply.github.com>

* chore(deps): update dependency gradle to v8.6 (#954)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update dependency org.junit.jupiter:junit-jupiter to v5.10.2 (#955)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Add PigZombieMock (#957)

* add PigZombieMock

* fix paper implementation && added descriptive names

* update unit tests

* remove field unused

* Fix WorkbenchInventoryMock#setMatrix (#960)

* chore(deps): update gradle/wrapper-validation-action action to v2 (#952)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update actions/setup-java action to v4 (#915)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update actions/cache action to v4 (#948)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* feat(world & server): Implement Spawn Limit API (#909)

* feat(world & server): Implement Spawn Limit API

* Fix merge issues

---------

Co-authored-by: Thorinwasher <ragnarlol96@gmail.com>

* Fix merge issues

---------

Co-authored-by: J3fftw <44972470+J3fftw1@users.noreply.github.com>
Co-authored-by: thelooter <evekolb2204@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Nirbose <62440475+Nirbose@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants