new ItemStack => ItemStack.of

[Intybyte]

Description

Use ItemStack.of instead of new ItemStack

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.

[github-actions]

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! 👀

[ybw0014]

this would break backward compatibility, add version check

[Intybyte]

this would break backward compatibility, add version check

i completely forgot about that, on it

[Intybyte]

Applied to everything beside test enviroment for now

[md5sha256]

LGTM

[Intybyte]

Maybe it is better if I move the StackResolver to the compatibility package, didn't know such a package exists already, should i call it VersionedItemStack?

[Intybyte]

Running with a local fork of MockBukkit