Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactoring: Simplify adding new configuration properties #2107

Merged
merged 2 commits into from
Jan 31, 2024
Merged

Refactoring: Simplify adding new configuration properties #2107

merged 2 commits into from
Jan 31, 2024

Commits on Jan 30, 2024

  1. Refactoring: Create interface for config initialization 

    This eliminates argument order mistakes, and also lets us add new inputs
without having to update every test.
    @henrymercer
    henrymercer committed Jan 30, 2024
    Configuration menu
    Copy the full SHA
    ec42edc View commit details
    Browse the repository at this point in the history

  2. Refactoring: Introduce common method to stub a config 

    This means that we don't need to update irrelevant test cases when we
add a new configuration property.
    @henrymercer
    henrymercer committed Jan 30, 2024
    Configuration menu
    Copy the full SHA
    d2e867f View commit details
    Browse the repository at this point in the history