Skip to content

Conversation

@ty-porter
Copy link
Contributor

@ty-porter ty-porter commented Nov 3, 2025

Required

  • Config migration feature
  • Happy path testing for the config migration feature
  • Rework validate_config.py:
    • Should be read-only (changes must come from a migration)
    • Should correctly associate and validate custom configs to its associated schema (ex. config.json and config.alt.json belong to the same config family referencing config.schema.json)

Nice to Have

  • Refactor -- centralize config path strings to a helper that associates configs in the same config family (avoid brittle string interpolation all over the codebase)

@WardBrian
Copy link
Member

I can give the config changes and migration code a once over soon if you’re hoping to do this release during the offseason still

@ty-porter
Copy link
Contributor Author

@WardBrian Sounds good. The migration code should be in a decent place. I will update the PR description for what else I'm planning before I consider this ready to release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants