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

Allow setting allow_dirty=False in the config file #183

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

Allow setting allow_dirty=False in the config file #183

wants to merge 4 commits into from

Commits on Jan 12, 2021

  1. Test various usages of allow_dirty in config 

    Currently, this one fails:
test_dirty_work_dir_with_allow_dirty_false_in_config_file_fails_with_warning
    @florisla
    florisla committed Jan 12, 2021
    Configuration menu
    Copy the full SHA
    4001d67 View commit details
    Browse the repository at this point in the history

  2. Corrected issues with the boolean operators in the config file being … 

    …treated as strings.
    @florisla
    Reynolds, AJ (ar380v) authored and florisla committed Jan 12, 2021
    Configuration menu
    Copy the full SHA
    926048a View commit details
    Browse the repository at this point in the history

  3. Fix typo 

    This test now passes:
test_dirty_work_dir_with_allow_dirty_false_in_config_file_fails_with_warning
    @florisla
    florisla committed Jan 12, 2021
    Configuration menu
    Copy the full SHA
    7062a18 View commit details
    Browse the repository at this point in the history

  4. Move list of FLAG_ARGUMENTS to a constant

    @florisla
    florisla committed Jan 12, 2021
    Configuration menu
    Copy the full SHA
    5ad4534 View commit details
    Browse the repository at this point in the history