Skip to content

Add state to Geoscanner return#806

Open
Da5h1n wants to merge 3 commits intoIntelligenceModding:dev/1.21.1from
Da5h1n:dev/1.21.1
Open

Add state to Geoscanner return#806
Da5h1n wants to merge 3 commits intoIntelligenceModding:dev/1.21.1from
Da5h1n:dev/1.21.1

Conversation

@Da5h1n
Copy link

@Da5h1n Da5h1n commented Feb 4, 2026

PLEASE READ THE GUIDELINES BEFORE MAKING A CONTRIBUTION

  • Please check if the PR fulfills these requirements
  • The commit message are well described
  • Docs have been added / updated (for features or maybe bugs which were noted). If not, please update the needed documentation here. Feel free to remove this check if you don't need it
  • All changes have fully been tested
  • What kind of change does this PR introduce? (Bug fix, feature, ...)
    Feature

  • What is the current behavior? (You can also link to an open issue here)
    Geoscanners do not return Blockstate

  • What is the new behavior (if this is a feature change)?
    Geoscanners can return Blockstate

  • Does this PR introduce a breaking change? (What changes might users need to make in their scripts due to this PR?)
    No

  • Other information:
    (my first time contributing on github, hope i done everything correctly)

@zyxkad
Copy link
Collaborator

zyxkad commented Feb 4, 2026

Code generally looks good, just need some formatting & caching.

However, why is state needed in the result? What's the use example?

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

Build Preview

badge

You can find files attached to the below linked Workflow Run URL (Logs).

Name Link
Commit 7642e2d
Logs https://github.com/IntelligenceModding/AdvancedPeripherals/actions/runs/21688143372
Jar Files AdvancedPeripherals PR 806
Expires At 2026-05-05T20:58:26Z

@SirEndii
Copy link
Member

SirEndii commented Feb 4, 2026

Check build log for checkstyle issues and resolve them

Also could you answer zyxkads question about the use case

@Da5h1n
Copy link
Author

Da5h1n commented Feb 4, 2026

that should be fixed now
and im currently using this for a 3D dashboard im working on:
image

probably not the best use case but it can probably help with waterlogged blocks and mining turtles

@Da5h1n
Copy link
Author

Da5h1n commented Feb 4, 2026

ive tested it both on turtles and as a peripheral and it works fine

@SirEndii SirEndii requested a review from zyxkad February 4, 2026 22:54
@SirEndii SirEndii added enhancement New feature or request 1.21.1 labels Feb 4, 2026
@SirEndii SirEndii moved this to Needs testing/review in Advanced Peripherals Feb 4, 2026
@SirEndii SirEndii added this to the 0.7.x milestone Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.21.1 enhancement New feature or request

Projects

Status: Needs testing/review

Development

Successfully merging this pull request may close these issues.

3 participants