Skip to content

Improve the plugin for 0.2.0 Release#68

Open
Zoriot wants to merge 40 commits intomainfrom
build
Open

Improve the plugin for 0.2.0 Release#68
Zoriot wants to merge 40 commits intomainfrom
build

Conversation

@Zoriot
Copy link
Collaborator

@Zoriot Zoriot commented Jan 10, 2026

This PR contains major changes to the plugin, making it ready for the next release.
We also switched to Gradle now, which allows us to do more things in the future & improves building performance.
Build's now have a distinct name to tracker better what version is currently running.

@Zoriot Zoriot added this to the Release v0.2.0 milestone Jan 10, 2026
@Zoriot Zoriot requested a review from Copilot January 10, 2026 21:05
@Zoriot Zoriot self-assigned this Jan 10, 2026
@Zoriot Zoriot added 🚨 high priority This currently has high priority. 🐻 medium size This is a medium issue. labels Jan 10, 2026
@Zoriot Zoriot linked an issue Jan 10, 2026 that may be closed by this pull request

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

MineFact and others added 21 commits February 3, 2026 14:56
@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Qodana for JVM

349 new problems were found

Inspection name Severity Problems
Usage of API marked for removal 🔴 Failure 1
Deprecated API usage 🔶 Warning 155
@NotNull/@Nullable problems 🔶 Warning 24
Lombok @Getter may be used 🔶 Warning 23
Nullability and data flow problems 🔶 Warning 20
Call to 'printStackTrace()' 🔶 Warning 20
Unchecked warning 🔶 Warning 18
Lombok @Setter may be used 🔶 Warning 14
Redundant type cast 🔶 Warning 10
Call to 'Arrays.asList()' with too few arguments 🔶 Warning 8
Field can be local variable 🔶 Warning 7
Field may be 'final' 🔶 Warning 5
Javadoc declaration problems 🔶 Warning 4
Unstable API Usage 🔶 Warning 4
Declaration has problems in Javadoc references 🔶 Warning 3
Redundant local variable 🔶 Warning 3
Access static member via instance reference 🔶 Warning 2
C-style array declaration 🔶 Warning 2
Constant values 🔶 Warning 2
Stream API call chain can be simplified 🔶 Warning 2
'size() == 0' can be replaced with 'isEmpty()' 🔶 Warning 2
Concatenation with empty string 🔶 Warning 2
Unnecessary semicolon 🔶 Warning 2
Unnecessary call to 'toString()' 🔶 Warning 2
Redundant 'close()' 🔶 Warning 1
Redundant 'String' operation 🔶 Warning 1
Commented out code ◽️ Notice 8
Non-strict inequality '>=' or '<=' can be replaced with '==' ◽️ Notice 3
Method can be extracted ◽️ Notice 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

DonatiFilippo
DonatiFilippo previously approved these changes Feb 7, 2026
Copy link
Member

@DonatiFilippo DonatiFilippo left a comment

Choose a reason for hiding this comment

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

Code was reviewed and most functionality were tested in game.
Approved for merge

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

House gen is currently broken on 1.21.4
https://pastebin.com/kb0ZQV8C

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@DonatiFilippo please test that again

@Zoriot Zoriot requested a review from DonatiFilippo February 13, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚨 high priority This currently has high priority. 🐻 medium size This is a medium issue.

Projects

None yet

4 participants