Skip to content

The rename: Optional is now Integrated#70

Merged
onebeastchris merged 1 commit intomasterfrom
feature/integrated
Nov 16, 2025
Merged

The rename: Optional is now Integrated#70
onebeastchris merged 1 commit intomasterfrom
feature/integrated

Conversation

@onebeastchris
Copy link
Member

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR renames the project from "GeyserOptionalPack" to "GeyserIntegratedPack" to reflect that the pack is now integrated into Geyser by default rather than being optional.

  • Complete rename of package names from org.geysermc.optionalpack to org.geysermc.integratedpack
  • Update of all file paths from optionalpack to integratedpack
  • Update of manifest UUIDs and version numbers for the new integrated pack

Reviewed Changes

Copilot reviewed 19 out of 88 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/main/resources/optionalpack/manifest.json Removed old optional pack manifest
src/main/resources/integratedpack/manifest.json Added new integrated pack manifest with updated name, description, and UUIDs
src/main/java/org/geysermc/integratedpack/*.java Renamed all Java packages and updated class/file references
build.gradle.kts Updated group ID and main class references
settings.gradle.kts Updated root project name
README.md Updated documentation to reflect integrated pack status
.github/workflows/build.yml Updated CI/CD workflow references
developer_documentation.md Updated all technical documentation references
Comments suppressed due to low confidence (1)

src/main/java/org/geysermc/integratedpack/renderers/SweepAttackRenderer.java:31

  • The comment should be updated to consistently use "IntegratedPack" instead of "optional pack" for clarity.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@onebeastchris onebeastchris merged commit 982fa6c into master Nov 16, 2025
7 checks passed
@onebeastchris onebeastchris deleted the feature/integrated branch November 17, 2025 01:33
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