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

Small refactor #42

Merged
merged 5 commits into from
Jul 13, 2022
Merged

Small refactor #42

merged 5 commits into from
Jul 13, 2022

Commits on Jul 13, 2022

  1. Small refactor to config 

    - Pull config stuff out of main, because it will likely keep growing
  quite a bit over time. The config system will actually be quite
  complex.
- Add a few more warning messages to handle json parsing stuff.
- Add a few unit tests.
    @johndgiese
    johndgiese committed Jul 13, 2022
    Configuration menu
    Copy the full SHA
    71545ff View commit details
    Browse the repository at this point in the history

  2. Add links to commandline arguments

    @johndgiese
    johndgiese committed Jul 13, 2022
    Configuration menu
    Copy the full SHA
    3ad4d81 View commit details
    Browse the repository at this point in the history

  3. Clean out unused function

    @johndgiese
    johndgiese committed Jul 13, 2022
    Configuration menu
    Copy the full SHA
    8dbf6a2 View commit details
    Browse the repository at this point in the history

  4. Address comments

    @johndgiese
    johndgiese committed Jul 13, 2022
    Configuration menu
    Copy the full SHA
    ed6cf7f View commit details
    Browse the repository at this point in the history

  5. Improve the error message upon badly formatted json 

    This was helpful to me as I tried this out!
    @johndgiese
    johndgiese committed Jul 13, 2022
    Configuration menu
    Copy the full SHA
    8ced94f View commit details
    Browse the repository at this point in the history