1.21.1 version

[TheekshanaA]

No description provided.

[Copilot]

Copilot reviewed 16 out of 21 changed files in this pull request and generated no comments.

Files not reviewed (5)

• fabric/build.gradle: Language not supported
• fabric/src/main/resources/fabric.mod.json: Language not supported
• forge/build.gradle: Language not supported
• gradle.properties: Language not supported
• gradle/wrapper/gradle-wrapper.properties: Language not supported

Comments suppressed due to low confidence (3)

common/src/main/java/io/github/jumperonjava/blockatlas/mixin/GameMenuScreenMixin.java:36

• Using @Local with ordinal 0 could be brittle if the method's local variable ordering changes. Consider verifying that the correct variable is captured or switching to a more explicit capture mechanism if available.

void addServersButton(CallbackInfo ci, @Local(ordinal = 0) GridWidget.Adder adder){

common/src/main/java/io/github/jumperonjava/blockatlas/gui/ServerScreen.java:171

• The added null check for the client may silently ignore situations where the client is unexpectedly null. Ensure that this behavior is intentional and consider logging or handling the null case more explicitly.

if (client == null) return;

common/src/main/java/io/github/jumperonjava/blockatlas/gui/elements/LazyUrlTexture.java:22

• There is an extraneous semicolon at the end of the ThreadPoolExecutor declaration. Removing the extra semicolon will improve code clarity.

private static ThreadPoolExecutor LAZY_DOWNLOAD_ASYNC = new ScheduledThreadPoolExecutor(4, (new ThreadFactoryBuilder()).setNameFormat("Icon download pool #%d").setDaemon(true).build());;