Skip to content

removal: unused warnings#12478

Merged
kennytv merged 2 commits into
PaperMC:mainfrom
SpigotRCE:disconnect-player-on-illegal-block-break
Apr 25, 2025
Merged

removal: unused warnings#12478
kennytv merged 2 commits into
PaperMC:mainfrom
SpigotRCE:disconnect-player-on-illegal-block-break

Conversation

@SpigotRCE
Copy link
Copy Markdown
Contributor

No description provided.

@SpigotRCE SpigotRCE requested a review from a team as a code owner April 25, 2025 06:13
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Apr 25, 2025
@NonSwag
Copy link
Copy Markdown
Contributor

NonSwag commented Apr 25, 2025

I think kicking the player is a bad idea and the logic should be untouched
Players with bad ping or laggy servers might cause inventory desync so this would kick legit players for nothing

@SpigotRCE SpigotRCE closed this Apr 25, 2025
@SpigotRCE SpigotRCE deleted the disconnect-player-on-illegal-block-break branch April 25, 2025 08:13
@github-project-automation github-project-automation Bot moved this from Awaiting review to Closed in Paper PR Queue Apr 25, 2025
@SpigotRCE SpigotRCE restored the disconnect-player-on-illegal-block-break branch April 25, 2025 08:13
@SpigotRCE SpigotRCE reopened this Apr 25, 2025
@SpigotRCE SpigotRCE changed the title fix: disconnect player when illegal blocks has been placed removal: unused warnings Apr 25, 2025
@github-project-automation github-project-automation Bot moved this from Closed to Awaiting final testing in Paper PR Queue Apr 25, 2025
@kennytv kennytv merged commit a211ac2 into PaperMC:main Apr 25, 2025
3 checks passed
@github-project-automation github-project-automation Bot moved this from Awaiting final testing to Merged in Paper PR Queue Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

3 participants