Skip to content

Conversation

@chsami
Copy link
Owner

@chsami chsami commented Feb 8, 2026

No description provided.

ZeynelTok and others added 10 commits February 3, 2026 19:49
This is controlled by net.ipv4.ping_group_range and appears to be disabled by default on many common Linux distros
Rev 236 changed menu action descriptor from "(IIIIIILjava/lang/String;Ljava/lang/String;IIB)V" -> "(IIIIIILjava/lang/String;Ljava/lang/String;III)V"
# Conflicts:
#	gradle.properties
Update MENU_ACTION_DESCRIPTOR_VANILLA signature
@chsami chsami merged commit 9cb2186 into main Feb 8, 2026
2 of 3 checks passed
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 8, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

This PR bumps project and microbot versions in gradle.properties, adds new game-constant identifiers across multiple runelite-api classes (including NpcID, ItemID, AnimationID, DBTableID, InterfaceID, and VarbitID), updates a cryptic clue location coordinate, modifies a method descriptor used in reflection-based menu action resolution, adjusts TCP ping control flow logic on non-Windows platforms, and refactors BankMainBuild script control flow with new conditional branches and label reorganization.

Possibly related PRs

✨ Finishing touches
  • 📝 Generate docstrings

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ast-grep (0.40.5)
runelite-api/src/main/java/net/runelite/api/NpcID.java
runelite-api/src/main/java/net/runelite/api/NullItemID.java
runelite-api/src/main/java/net/runelite/api/gameval/AnimationID.java
  • 5 others

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants