Skip to content

Comments

2.0.9#1530

Merged
chsami merged 2 commits intomainfrom
development
Sep 21, 2025
Merged

2.0.9#1530
chsami merged 2 commits intomainfrom
development

Conversation

@chsami
Copy link
Owner

@chsami chsami commented Sep 21, 2025

removed caching references

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 21, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

The change updates the microbot.version in the client’s POM, adjusts a Microbot boolean flag’s visibility to public, and removes select cache-related imports. Rs2Player now retrieves quest and skill data directly via the Microbot client and client thread instead of cache utilities, without altering public method signatures. Rs2Magic cleans up an unused cache import. Rs2Widget modifies hasWidgetText checks to require that matching widgets are not hidden, leaving signatures intact. Overall, the codebase reduces reliance on cache classes, standardizes imports, and introduces a visibility constraint in widget text presence checks.

✨ Finishing touches
  • 📝 Generate Docstrings

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0f62de9 and 4aaf330.

📒 Files selected for processing (5)
  • runelite-client/pom.xml (1 hunks)
  • runelite-client/src/main/java/net/runelite/client/plugins/microbot/Microbot.java (1 hunks)
  • runelite-client/src/main/java/net/runelite/client/plugins/microbot/util/magic/Rs2Magic.java (0 hunks)
  • runelite-client/src/main/java/net/runelite/client/plugins/microbot/util/player/Rs2Player.java (4 hunks)
  • runelite-client/src/main/java/net/runelite/client/plugins/microbot/util/widget/Rs2Widget.java (3 hunks)

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.

1 participant