cabaletta / baritone

google maps for block game
GNU Lesser General Public License v3.0
7.27k stars 1.45k forks source link

Make allowBreakAnyways an array of blocks #4386

Closed brentspine closed 5 months ago

brentspine commented 5 months ago

Describe your suggestion

The allowBreakAnyways option allows the client to break a certain block, even if allowBreak is set to false. My suggestion is, that it allows multiple blocks in this option, instead of only one singular.

Example

I want the bot to mine multiple types of ores in a cave, but don't want to allow breaking stone. For this I would set allowBreak to false and set allowBreakAnyways to diamond ore. Now it will only break diamond ores and no stone. But what if there is also deep slate diamond ore. We would need to decide which one we want. This is why I suggest we allow multiple blocks as the value of allowBreakAnyways

Final checklist

ZacSharp commented 5 months ago

You can already separate multiple blocks by commas (not spaces) to do this. In your case that would be #set allowBreakAnyway diamon_ore,deepslate_diamon_ore, though note that mining doesn't work if it can't walk into the positions where the target block is.

Also yes there are people reading the checklist. Please make sure your issue is well formed by removing the extraneous line(s) so you can check the missing box :smile: