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

Update to 1.21 #4248

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Update to 1.21 #4248

wants to merge 5 commits into from

Conversation

WalshyDev
Copy link
Member

Description

Updates to 1.21 - big shout out to MockBukkit team and @Intybyte for the work on tests

Proposed changes

Updated Slimefun code to support 1.21

Related Issues (if applicable)

N/A

Checklist

  • I have fully tested the proposed changes and promise that they will not break everything into chaos.
  • I have also tested the proposed changes in combination with various popular addons and can confirm my changes do not break them.
  • I have made sure that the proposed changes do not break compatibility across the supported Minecraft versions (1.16.* - 1.20.*).
  • I followed the existing code standards and didn't mess up the formatting.
  • I did my best to add documentation to any public classes or methods I added.
  • I have added Nonnull and Nullable annotations to my methods to indicate their behaviour for null values
  • I added sufficient Unit Tests to cover my code.

@WalshyDev WalshyDev requested review from a team as code owners October 1, 2024 00:23
Copy link
Contributor

github-actions bot commented Oct 1, 2024

Pro Tip!
You can help us label your Pull Requests by using the following branch naming convention next time you create a pull request. ❤️

Branch naming convention Label
feature/** 🎈 Feature
fix/** ✨ Fix
chore/** 🧹 Chores
api/** 🔧 API
performance/** 💡 Performance Optimization
compatibility/** 🤝 Compatibility

If your changes do not fall into any of these categories, don't worry. You can just ignore this message in that case! 👀

Copy link
Contributor

github-actions bot commented Oct 1, 2024

Slimefun preview build

A Slimefun preview build is available for testing!
Commit: 99cdc01

https://preview-builds.walshy.dev/download/Slimefun/4248/99cdc018

Note: This is not a supported build and is only here for the purposes of testing.
Do not run this on a live server and do not report bugs anywhere but this PR!

@WalshyDev
Copy link
Member Author

This still needs major testing

@WalshyDev WalshyDev added 🎯 Needs testing This Issue needs to be tested by our team to see if it can be reproduced. 🤝 Compatibility This Pull Request introduces a compatibility feature. 🎮 Game Update This pull request adds compatibility with a new version of Minecraft. labels Oct 1, 2024
pom.xml Show resolved Hide resolved
@WalshyDev WalshyDev force-pushed the walshy/mc-1.21 branch 4 times, most recently from 41f9139 to afaf3aa Compare October 1, 2024 01:03
Comment on lines 109 to 111
ItemStack item = new CustomItemStack(Material.BEACON, "&cItem Test");
String id = "IS_ITEM_TEST";
SlimefunItem sfItem = TestUtilities.mockSlimefunItem(plugin, id, item);
SlimefunItem sfItem = TestUtilities.mockSlimefunItem(plugin, id, new CustomItemStack(Material.BEACON, "&cItem Test"));
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the point of this?

Copy link
Contributor

Choose a reason for hiding this comment

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

I think its testing that SlimefunItem#register should change the underlying ItemStack in some way, thus sfItem#isItem should return false even if the twoItemStacks were the same when we started.

@Boomer-1
Copy link

Boomer-1 commented Oct 1, 2024

testing produced multiple errors immediately which stopped any further testing. I've linked them from the contributors channel on discord. https://discord.com/channels/565557184348422174/565570276038017044/1290525215461216266

Copy link

sonarcloud bot commented Oct 1, 2024

@Stis780
Copy link

Stis780 commented Oct 22, 2024

testing produced multiple errors immediately which stopped any further testing. I've linked them from the contributors channel on discord. https://discord.com/channels/565557184348422174/565570276038017044/1290525215461216266

Is there 1.21 build available? where? ty mate

@Boomer-1
Copy link

testing produced multiple errors immediately which stopped any further testing. I've linked them from the contributors channel on discord. https://discord.com/channels/565557184348422174/565570276038017044/1290525215461216266

Is there 1.21 build available? where? ty mate

no. this is the work being done to get 1.21 up and running.

@sruly09
Copy link

sruly09 commented Oct 22, 2024

when will 1.21 version come out?

@Intybyte
Copy link
Contributor

slimefun-updated

@Boomer-1
Copy link

when all the changes that mojang and paper put in place are addressed and fixed. feel free to help

@Boomer-1
Copy link

lets try to keep comments not related to the PR on Discord please

@Outfit8TSB

This comment has been minimized.

@Intybyte
Copy link
Contributor

Please stop posting anything here, we already know what we need to fix it just requires time and patience and lots of changes, also you will not recive support for your own fork of the plugin.

@JustinDevB
Copy link

Do we ever plan on updating this? Or are we just going to let minecraft continue to update and abandon this project

@ybw0014
Copy link
Contributor

ybw0014 commented Nov 4, 2024

@JustinDevB Please do not add noisy comments to this PR, thank you. Questions go on Discord only.

@JustinDevB
Copy link

@JustinDevB Please do not add noisy comments to this PR, thank you. Questions go on Discord only.

All that happens there is people get warned or muted for asking about the progress. Seems to be a topic nobody is allowed to discuss anywhere.

@Lamby777
Copy link

Lamby777 commented Nov 4, 2024

All that happens there is people get warned or muted for asking about the progress. Seems to be a topic nobody is allowed to discuss anywhere.

can confirm, i was banned for no reason (i don't even talk there)

@Outfit8TSB
Copy link

Outfit8TSB commented Nov 7, 2024

Please stop posting anything here, we already know what we need to fix it just requires time and patience and lots of changes, also you will not recive support for your own fork of the plugin.

Look, it's not fork(and it's git clone) and it's resolved by #4253. Ok? I marked it as resolved for you.

@Outfit8TSB Outfit8TSB mentioned this pull request Nov 9, 2024
5 tasks
Copy link

sonarcloud bot commented Nov 9, 2024

@RyuguDev

This comment was marked as off-topic.

@Boomer-1
Copy link

You're leaving an entire bubble out in the rain, you can keep talking about it, but slimefun will soon die for all the servers that once offered it!

please leave unproductive comments like these out of the PR discussion. since you clearly didn't look at what's going on and seen the progress, you aren't helping anyone.

@sruly09
Copy link

sruly09 commented Dec 15, 2024

yay the checklist is shrinking

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤝 Compatibility This Pull Request introduces a compatibility feature. 🎮 Game Update This pull request adds compatibility with a new version of Minecraft. 🎯 Needs testing This Issue needs to be tested by our team to see if it can be reproduced.
Projects
None yet
Development

Successfully merging this pull request may close these issues.