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: Use tryJust instead of pattern guards #71

Closed
wants to merge 4 commits into from
Closed

refactoring: Use tryJust instead of pattern guards #71

wants to merge 4 commits into from

Commits on May 22, 2023

  1. Add redundant waitForProcess to make duplication more obvious 

    It can never run because of the throwIO before it
    @tomjaguarpaw
    tomjaguarpaw committed May 22, 2023
    Configuration menu
    Copy the full SHA
    1cb63a9 View commit details
    Browse the repository at this point in the history

  2. Remove duplication between branches

    @tomjaguarpaw
    tomjaguarpaw committed May 22, 2023
    Configuration menu
    Copy the full SHA
    ff029bc View commit details
    Browse the repository at this point in the history

  3. Pull out terminateProcess wrapper function

    @tomjaguarpaw
    tomjaguarpaw committed May 22, 2023
    Configuration menu
    Copy the full SHA
    8a0026a View commit details
    Browse the repository at this point in the history

Commits on May 23, 2023

  1. refactoring: Use tryJust instead of pattern guards

    @sol
    sol committed May 23, 2023
    Configuration menu
    Copy the full SHA
    cb67af7 View commit details
    Browse the repository at this point in the history