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

--check option, mirroring black --check #36

Merged
merged 4 commits into from
Aug 7, 2020
Merged

--check option, mirroring black --check #36

merged 4 commits into from
Aug 7, 2020

Commits on Aug 6, 2020

  1. --check option, mirroring black --check

    @akaihola
    akaihola committed Aug 6, 2020
    Configuration menu
    Copy the full SHA
    ae4334d View commit details
    Browse the repository at this point in the history

  2. Unit tests for --check and return values

    @akaihola
    akaihola committed Aug 6, 2020
    Configuration menu
    Copy the full SHA
    e775077 View commit details
    Browse the repository at this point in the history

  3. Make Pylint and Flake8 happy 

    (only for regions we touched in this branch)
    @akaihola
    akaihola committed Aug 6, 2020
    Configuration menu
    Copy the full SHA
    d32c5c9 View commit details
    Browse the repository at this point in the history

  4. Refactor acting on reformat results into functions 

    `format_edited_parts()` now yields the proposed changes for each file,
and two separate functions display diffs or modify files.  This way
high-level control remains high in the call stack and the
`--diff`/`--check` flags don't have to travel far in the function
calls.

Thanks for review @samoylovfp!
    @akaihola
    akaihola committed Aug 6, 2020
    Configuration menu
    Copy the full SHA
    6ee2a17 View commit details
    Browse the repository at this point in the history