Skip to content

[ci skip] Clarify BlockFadeEvent#getNewState javadocs#12250

Merged
notTamion merged 2 commits into
PaperMC:mainfrom
Github-Citizen:For_PR
Mar 9, 2025
Merged

[ci skip] Clarify BlockFadeEvent#getNewState javadocs#12250
notTamion merged 2 commits into
PaperMC:mainfrom
Github-Citizen:For_PR

Conversation

@Github-Citizen
Copy link
Copy Markdown
Contributor

Old wording of:

"Gets the state of the block that will be fading, melting or disappearing"

Sounds as if it will return the same block as .getBlock()

For example, Ice melts and become water. Current wording of "Gets ... the block that will be ... melting" would mean ice. But the method would return water.

Old wording of:

"Gets the state of the block that will be fading, melting or disappearing"

Sounds as if it will return the same block as .getBlock()

For example, Ice melts and become water. Current wording of "Gets ... the block that will be ... melting" would mean ice. But the method would return water.
@Github-Citizen Github-Citizen requested a review from a team as a code owner March 9, 2025 01:31
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Mar 9, 2025
@notTamion notTamion changed the title Update BlockFadeEvent.java .getNewBlock() Java Docs [ci skip] Clarify BlockFadeEvent#getNewState javadocs Mar 9, 2025
@github-project-automation github-project-automation Bot moved this from Awaiting review to Awaiting final testing in Paper PR Queue Mar 9, 2025
@notTamion notTamion merged commit f4f2755 into PaperMC:main Mar 9, 2025
@github-project-automation github-project-automation Bot moved this from Awaiting final testing to Merged in Paper PR Queue Mar 9, 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