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

UmbrellaDish patch importable fido 1 #149

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

UmbrellaDish patch importable fido 1 #149

wants to merge 2 commits into from

Commits on Feb 1, 2019

  1. Make fido importable as a module 

    *Not tested, yet* Take it as an issue with source code attachment ready to merge if error-free.

Moved argument processing and sys.argv checks out of main() into `if __name__ == '__main__'` clause.

Applied dest='other_name' for certain names that have reserved meaning in python or which are to short to be meaningful. `main()` accepts directly passed keyword arguments (in theory they may be positional, but rather don't), so the caller does not need argparse.
    @UmbrellaDish
    UmbrellaDish authored Feb 1, 2019
    Configuration menu
    Copy the full SHA
    c4f0343 View commit details
    Browse the repository at this point in the history

  2. Fixes; renamed line_processor argument -> handle_matches 

    Spotted more left-overs to fix after lunch
    @UmbrellaDish
    UmbrellaDish authored Feb 1, 2019
    Configuration menu
    Copy the full SHA
    2aa8452 View commit details
    Browse the repository at this point in the history