Skip to content

Commit

Permalink
Try to unify our config files a bit
Browse files Browse the repository at this point in the history
I've tried so many rewrites of the config system over the last few
months, in an attempt to get started on #1727. All of them stink, so
this is an attempt to apply some of the cleanup.

 - Move some of the common logic into ConfigFile. This means we now
   store more information ourselves for Forge, rather than reading it
   out of the ForgeConfigSpec.

 - Don't include the Range/Allowed keys in the translation key. This was
   mostly there because of how we read comments from Forge, but it never
   made much sense.

 - Remove our separate Trie structure, and just encode the tree as part
   of the children of a Group.
  • Loading branch information
SquidDev committed Nov 24, 2024
1 parent 1963e01 commit ea670cc
Show file tree
Hide file tree
Showing 6 changed files with 217 additions and 301 deletions.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit ea670cc

Please sign in to comment.