chore: v0.12.0 pre-release review and cleanup - #105
Merged
Conversation
- Remove unused MessageUtil import from AdminSubCommand and AdminEconomyHandler - Remove unused UUID import from TreasuryCommandHandler - Make SentryIntegration.initialized volatile for thread safety - Replace System.err.println in EventBus with ErrorHandler.report
Convert Logger-only catch blocks across the codebase to use ErrorHandler.report(), which logs to console AND sends to Sentry. Covers platform, managers, integrations, storage, worldmap, GUI, migrations, territory, and update packages. Informational warnings (not in catch blocks) are intentionally left as Logger.warn calls.
- Add missing changelog entries: SimpleClaims/FactionsX importers, BetterMap compatibility, i18n localization, ocean claim visibility fix - Update README feature tables: importers, config version, admin GUI status, localization, GUI page count - Update all 12 docs/ version headers to 0.12.0 - Add SimpleClaims and FactionsX sections to data-import.md - Add V6->V7 and V7->V8 to migration table - Add admin GUI pages to gui.md (ConfigPage, BackupsPage, UpdatesPage) - Add ZoneMobClearManager to managers.md - Add BetterMap compatibility to integrations.md - Add import subcommands to commands.md admin tree
- Replace What's New section with v0.12.0 features (i18n, admin GUI pages, SimpleClaims/FactionsX importers, BetterMap, ocean fix) - Update data import references to include all 4 importers - Update GUI page count from 65+ to 70+ - Update JitPack version to v0.12.0
- Fix zone flag count: 50 -> 51 (verified against ZoneFlags.ALL_FLAGS) - Fix Integration flags category: was listing removed flags (command blocking, fluid spread, map visibility), now lists actual flags (gravestone access, show on map, essentials homes/warps/kits) - Fix GUI page count: 76 -> 70+ (67 actual page classes) - Add missing Core Features: faction economy with upkeep system, localization (10 languages) - Remove Sentry from public-facing description (dev-only feature) - Convert integrations section from nested lists to tables for easier maintenance as the list grows - Also fix zone flag count in README (50 -> 51)
- Fix HyperProtect-Mixin hook count: 27 -> 28 (verified SLOT_ constants) - Remove "Command blocking in zones" from README (not yet implemented)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Pre-release review and cleanup of the
releases/0.12.0branch before tagging. Covers code quality fixes, Sentry error routing, documentation accuracy, changelog completions, CurseForge description overhaul, and version bump.Code Quality (2 commits)
MessageUtilx2,UUID)SentryIntegration.initializedvolatile for thread safetySystem.err.printlninEventBuswithErrorHandler.report()ErrorHandler.report()(logs to console and sends to Sentry)Documentation (2 commits)
docs/version headers to 0.12.0data-import.mdgui.mdmanagers.mdintegrations.mdcommands.mdadmin treeCurseForge Description (2 commits)
ZoneFlags.INTEGRATION_FLAGS)Version Bump
build.gradleversion0.11.1→0.12.0Type of Change
Related Issues
Relates to the 7 PRs merged into
releases/0.12.0:Testing
./gradlew :HyperFactions:shadowJar— BUILD SUCCESSFULreleases/0.12.0, not introduced by this PRSystem.err.printlnore.printStackTrace()in runtime codeChecklist
Accuracy Verification
All CurseForge description claims verified against source code:
ZoneFlags.ALL_FLAGS(51 entries)gui/*/page/SLOT_constantsFactionCommand.javaConfigManager.javaSortModeenumRefreshModeenum