Skip to content

Remove the vendored Bukkit configuration API - #3718

Open
MattBDev wants to merge 6 commits into
refactor/v3/settings-recordsfrom
refactor/v3/remove-vendored-config
Open

MattBDev wants to merge 6 commits into
refactor/v3/settings-recordsfrom
refactor/v3/remove-vendored-config

Conversation

@MattBDev

@MattBDev MattBDev commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Overview

Removes the copy of Bukkit's configuration API that FAWE vendored under com.fastasyncworldedit.core.configuration: Configuration, ConfigurationSection, MemorySection, file.YamlConfiguration, the serialization package and their option classes (17 files, 3014 lines deleted, nothing added).

Its only user was Config, which the Settings records rewrite (#3703) removed, so nothing references these classes anymore.

FAWE carried a copy of Bukkit's configuration API (Configuration,
MemorySection, file.YamlConfiguration, the serialization package and
their option classes). Its only user was Config, which the records
rewrite of Settings removed, so nothing references it anymore.

Plugins that imported these classes can use Bukkit's own
org.bukkit.configuration, which they were copied from, or SnakeYAML.
@MattBDev
MattBDev requested a review from a team as a code owner November 6, 2026 04:32
@MattBDev
MattBDev added this pull request to stack #3704 November 6, 2026 04:36
FAWE carried a copy of Bukkit's configuration API (Configuration,
MemorySection, file.YamlConfiguration, the serialization package and
their option classes). Its only user was Config, which the records
rewrite of Settings removed, so nothing references it anymore.

Plugins that imported these classes can use Bukkit's own
org.bukkit.configuration, which they were copied from, or SnakeYAML.
@MattBDev
MattBDev force-pushed the refactor/v3/remove-vendored-config branch from 7e8b5e7 to 46fc013 Compare November 9, 2026 00:30
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