Skip to content

Deprecate existing matchPattern method#1002

Merged
Robotgiggle merged 6 commits into
FallingColors:mainfrom
beholderface:main
May 9, 2026
Merged

Deprecate existing matchPattern method#1002
Robotgiggle merged 6 commits into
FallingColors:mainfrom
beholderface:main

Conversation

@beholderface
Copy link
Copy Markdown
Contributor

No description provided.


protected boolean canOvercast() {
//bad deployer, no more infinite bloodcasting for you
if (this.caster.getClass() != ServerPlayer.class){
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This change seems unrelated (and the commit is from a year ago), did something weird happen with the commit history?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ack I thought that got deleted when I merged with the more recent commits

* order
* for a per-world pattern.
*/
@Deprecated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you add a description to the deprecated annotation, and update the javadoc to mention the new overload?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sorry, can you also add since= to the deprecated annotation? I should have been more clear about what I was asking

https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/lang/Deprecated.html#since()

@github-project-automation github-project-automation Bot moved this from 📋 Backlog to 🏗 In progress in Hex Casting Feb 19, 2026
Copy link
Copy Markdown
Member

@object-Object object-Object left a comment

Choose a reason for hiding this comment

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

LGTM

@Robotgiggle Robotgiggle added this pull request to the merge queue May 9, 2026
Merged via the queue into FallingColors:main with commit 40e824e May 9, 2026
8 checks passed
@github-project-automation github-project-automation Bot moved this from 🏗 In progress to ✅ Done in Hex Casting May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants