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

Add OO interface #41

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Add OO interface #41

wants to merge 5 commits into from

Commits on Oct 15, 2022

  1. Refactor to use object

    @zmughal
    zmughal committed Oct 15, 2022
    Configuration menu
    Copy the full SHA
    f534a13 View commit details
    Browse the repository at this point in the history

  2. Rename OS-checking functions to lowercase with underscore prefix

    @zmughal
    zmughal committed Oct 15, 2022
    Configuration menu
    Copy the full SHA
    28d0833 View commit details
    Browse the repository at this point in the history

  3. Extract getting path from $ENV{PATH} to method

    @zmughal
    zmughal committed Oct 15, 2022
    Configuration menu
    Copy the full SHA
    21ebfff View commit details
    Browse the repository at this point in the history

  4. Add constructor option fixed_paths 

    The name indicates that it will remain the same despite changes to
`$ENV{PATH}`.
    @zmughal
    zmughal committed Oct 15, 2022
    Configuration menu
    Copy the full SHA
    5b830c2 View commit details
    Browse the repository at this point in the history

  5. Use lowercase for implicit_current_dir option

    @zmughal
    zmughal committed Oct 15, 2022
    Configuration menu
    Copy the full SHA
    a9b7707 View commit details
    Browse the repository at this point in the history