Skip to content

Conversation

@johncsuti
Copy link
Contributor

Summary

This PR adds bStats metrics to InstantDeath and shades (relocates) the bStats classes to avoid classpath conflicts with other plugins.

Why

  • Helps track plugin adoption and Minecraft/server version usage to guide future maintenance.
  • Shading prevents collisions when multiple plugins bundle bStats.

What changed

  • Added bStats dependency + initialization in plugin startup.
  • Added Maven shading/relocation for org.bstats.
  • Minor compile fixes from the initial integration (imports/syntax/typo).

Notes for maintainer

  • bStats requires a plugin ID. If you prefer, I can change the ID with the ID from your own bStats dashboard.

@emipazd
Copy link
Owner

emipazd commented Dec 20, 2025

Will merge when Im on my pc

@johncsuti
Copy link
Contributor Author

Merge when you can then update line 23 in instantdeath.java with your own plugin ID

@emipazd emipazd marked this pull request as draft December 21, 2025 11:20
@emipazd emipazd marked this pull request as ready for review December 21, 2025 11:21
@emipazd
Copy link
Owner

emipazd commented Dec 21, 2025

ID 28460

@emipazd emipazd merged commit 5c835fa into emipazd:master Dec 21, 2025
1 check passed
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.

2 participants