Skip to content

Warn about unknown server property keys at run - #27

Merged
afrigon merged 4 commits into
mainfrom
invalid-property-warning
Sep 7, 2026
Merged

afrigon merged 4 commits into
mainfrom
invalid-property-warning

Conversation

@afrigon

@afrigon afrigon commented Sep 7, 2026 •

Copy link
Copy Markdown
Owner
  • mc run prints a warning for each key in the manifest's server > properties block that is not a known server property; the check happens in to_entries, against the base map it already builds from the ServerProperties defaults
  • the key is still written, since mods read their own keys from the file
  • unit test on the to_entries result, docs in the manifest reference and the mc run page

Closes #23

@afrigon afrigon self-assigned this Sep 7, 2026
@afrigon
afrigon merged commit 8799109 into main Sep 7, 2026
1 check passed
@afrigon
afrigon deleted the invalid-property-warning branch September 7, 2026 17:34
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.

add a warning when a invalid key is in the property array

1 participant