Skip to content

docs: describe MusicalInstruments repeat-note fix and coverage gate - #63

Merged
ryanbarlow97 merged 2 commits into
mainfrom
docs/musical-instruments-coverage
Sep 26, 2026
Merged

ryanbarlow97 merged 2 commits into
mainfrom
docs/musical-instruments-coverage

Conversation

@ryanbarlow97

@ryanbarlow97 ryanbarlow97 commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

Documents TF-Minecraft/MusicalInstruments#20:

  • Playing a note cancels the slot change so the server stays on slot 9, which lets the same note repeat on Paper. Slot changes other plugins cancel are ignored. Slot 9 is the reset slot and never plays a note; configs that map it are warned about.
  • Removes the description of the per-player repeating task, which the plugin no longer runs, and updates the architecture tree and class diagram.
  • /instruments give ignores case and drops the item when the inventory is full. v.minecraft:<material> resolves, and items that resolve to air are skipped.
  • Build section covers the 100% JaCoCo coverage gate and its model-data limitation.
  • InstrumentPlayEvent is not fired for slot changes another plugin cancelled.

Updates projects/MusicalInstruments/overview.md and README.md. The script unit tests and check-indexes.py pass.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
@ryanbarlow97
ryanbarlow97 merged commit c493c0b into main Sep 26, 2026
1 check passed
@ryanbarlow97
ryanbarlow97 deleted the docs/musical-instruments-coverage branch September 26, 2026 09:57
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