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

Improve usability #118

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Improve usability #118

wants to merge 3 commits into from

Commits on Nov 12, 2020

  1. Remove executable bit from source files 

    Having them with +x makes the shell autocomplete them when the dir
is in the user's $PATH, which is an unnecessary nuissance.
    @alanorth
    alanorth committed Nov 12, 2020
    Configuration menu
    Copy the full SHA
    62f5954 View commit details
    Browse the repository at this point in the history

  2. run_pipeline.py: Add executable bit 

    This allows the shell to autocomplete the name of the script like
any other executable when the SALSA dir is in the user's $PATH.
    @alanorth
    alanorth committed Nov 12, 2020
    Configuration menu
    Copy the full SHA
    832c02a View commit details
    Browse the repository at this point in the history

  3. Add python2 shebang to all Python scripts 

    These scripts are already executable due to their chmod +x, but they
are missing a shebang to tell the shell which interpreter is respons-
ible for executing them. The convention is to use the "env" command
to find an appropriate Python that may be in the user's environment
instead of hard coding a path, as the user might have a virtual env
loaded or might have Python 3 as the default "python" on the system.
    @alanorth
    alanorth committed Nov 12, 2020
    Configuration menu
    Copy the full SHA
    c8c26d2 View commit details
    Browse the repository at this point in the history