Skip to content

ServerTickManager#requestGameToSprint - Silence command like feedback#12220

Merged
Owen1212055 merged 3 commits into
PaperMC:mainfrom
ShaneBeee:shane/silent-tick-sprint
Mar 8, 2025
Merged

ServerTickManager#requestGameToSprint - Silence command like feedback#12220
Owen1212055 merged 3 commits into
PaperMC:mainfrom
ShaneBeee:shane/silent-tick-sprint

Conversation

@ShaneBeee
Copy link
Copy Markdown
Contributor

@ShaneBeee ShaneBeee commented Mar 3, 2025

This PR aims to silence the feedback from server sprinting via the API.

When using the API to request the server to sprint:
https://jd.papermc.io/paper/1.21.4/org/bukkit/ServerTickManager.html#requestGameToSprint(int)

The tick manager still provides feedback as if the vanilla /tick sprint command were run:
Screenshot 2025-03-02 at 10 45 31 PM

When using the API to do this task, the server shouldn't be responding as if a command were run.

This has driven me nuts since Bukkit added this, I just kept forgetting to report it and/or actually do something about it.
But nows my chance.

One thing I did think was someone out there using this API might actually like this feedback?!?!?
So I was thinking, should I make another ServerTickManager#requestGameToSprint method with an option to silence?
Looking for feedback!

@ShaneBeee ShaneBeee requested a review from a team as a code owner March 3, 2025 06:51
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Mar 3, 2025
@github-project-automation github-project-automation Bot moved this from Awaiting review to Awaiting final testing in Paper PR Queue Mar 8, 2025
@Owen1212055 Owen1212055 merged commit 34c794d into PaperMC:main Mar 8, 2025
@github-project-automation github-project-automation Bot moved this from Awaiting final testing to Merged in Paper PR Queue Mar 8, 2025
@ShaneBeee ShaneBeee deleted the shane/silent-tick-sprint branch March 9, 2025 00:06
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.

6 participants