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.

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

Factor out the injectorFactory #1202

Merged
merged 3 commits into from Jan 12, 2022
Merged

Commits on Jan 9, 2022

  1. Factor out the injectorFactory

    This tiny refactor is just some incidental cleanup on the CLI plumbing.
    
    - It removes the `injectorFacotry`
    - It reorders the arguments to `handleExceptions`
    
    In the current state of the code, the injectorFactory looks like it's
    just adding unhelpful indirection. Removing it allows us to save about
    10 lines of code and remove an unreachable throw.
    
    The arguments to `handleExceptions` are reordered to reflect the the
    underlying logic better. I.e., we call the `runner` on the `injector`
    and `cmd`, so the `runner` is now supplied first.
    Shon Feder committed Jan 9, 2022
    Configuration menu
    Copy the full SHA
    315bebf View commit details
    Browse the repository at this point in the history

Commits on Jan 10, 2022

  1. Configuration menu
    Copy the full SHA
    1a72f22 View commit details
    Browse the repository at this point in the history

Commits on Jan 12, 2022

  1. Configuration menu
    Copy the full SHA
    c99e9c1 View commit details
    Browse the repository at this point in the history