Skip to content

Conversation

@theminecoder
Copy link
Contributor

Add the ability to load plugins for directories other than the configured main plugin directory. Useful in container like contexts with features like Image Volumes

Plugins are still created with the main plugin directory as the data directory so this should be basically transparent.

@theminecoder theminecoder requested a review from a team as a code owner December 27, 2025 09:02
@github-project-automation github-project-automation bot moved this to Awaiting review in Paper PR Queue Dec 27, 2025
Copy link
Contributor

@lynxplay lynxplay left a comment

Choose a reason for hiding this comment

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

Right now, this automatically creates all specified directories, which is pretty annoying for what it is.

Could we alternatively create a secondary instance of the DirectoryProviderSource that is configured to not construct the directories it is preparing?

Should be a single field + ctor + static field.

@github-project-automation github-project-automation bot moved this from Awaiting review to Changes required in Paper PR Queue Dec 27, 2025
Copy link
Member

@Owen1212055 Owen1212055 left a comment

Choose a reason for hiding this comment

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

merger should cleanup style

@lynxplay lynxplay merged commit 93e81bf into PaperMC:main Jan 4, 2026
4 checks passed
@github-project-automation github-project-automation bot moved this from Changes required to Merged in Paper PR Queue Jan 4, 2026
@lynxplay
Copy link
Contributor

lynxplay commented Jan 4, 2026

Let me know if you need this backported to 1.21.10

@theminecoder
Copy link
Contributor Author

Personally I dont need it but not against if others want it

@theminecoder theminecoder deleted the kube-ready-plugin-dirs branch January 4, 2026 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

6 participants