Skip to content

Add cartography table plugin (lint-fixed version of #3778)#3826

Open
plainprince wants to merge 1 commit intoPrismarineJS:masterfrom
plainprince:fix/cartography-table-lint
Open

Add cartography table plugin (lint-fixed version of #3778)#3826
plainprince wants to merge 1 commit intoPrismarineJS:masterfrom
plainprince:fix/cartography-table-lint

Conversation

@plainprince
Copy link
Contributor

Summary

  • Rebased version of Added ability to use cartography table #3778 by @AboveAphid with lint fixes applied
  • Adds cartography table plugin (lib/plugins/cartography_table.js) enabling bot interaction with cartography tables (put/take maps, modifiers, and outputs)
  • Registers the plugin in lib/loader.js and documents the API in docs/api.md

Changes from #3778

Fixed all standard lint errors in cartography_table.js:

  • Added missing spaces before function parentheses (space-before-function-paren)
  • Removed blank line padding inside block (padded-blocks)

Note on 1.13.2 CI failure

The useChests test failure on 1.13.2 (27 !== 305) in #3778 is a pre-existing flaky test unrelated to these changes -- master CI passes consistently.

Test plan

  • npm run lint passes locally
  • CI lint check passes
  • All Minecraft version tests pass (cartography table changes are additive and don't affect existing tests)

Made with Cursor

Based on PrismarineJS#3778 by @AboveAphid. Adds ability to use cartography tables
(put/take maps, modifiers, and outputs). Fixes lint errors
(space-before-function-paren, padded-blocks) that caused CI to fail.

Co-authored-by: Cursor <[email protected]>
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