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

Format subscriptions in a PEP-8 compliant way #178

Merged
merged 4 commits into from
May 1, 2018
Merged

Format subscriptions in a PEP-8 compliant way #178

merged 4 commits into from
May 1, 2018

Commits on Apr 29, 2018

  1. Format subscriptions in a PEP-8 compliant way 

    Fixes #157
    @zsol
    zsol committed Apr 29, 2018
    Configuration menu
    Copy the full SHA
    1c8a620 View commit details
    Browse the repository at this point in the history

  2. fix type errors and ignore E203

    @zsol
    zsol committed Apr 29, 2018
    Configuration menu
    Copy the full SHA
    8f6b504 View commit details
    Browse the repository at this point in the history

  3. Address code review comments: 

    - Make `append` trivial again!
- move lsqb hunting into `BracketTracker`
- Consider expressions complex if they are children of `test`
- README section rename and typo fix

In two cases I still return `SPACE` explicitly in `whitespace()`. See comments.
    @zsol
    zsol committed Apr 29, 2018
    Configuration menu
    Copy the full SHA
    85ee8b8 View commit details
    Browse the repository at this point in the history

Commits on Apr 30, 2018

  1. style

    @zsol
    zsol committed Apr 30, 2018
    Configuration menu
    Copy the full SHA
    ae3b6da View commit details
    Browse the repository at this point in the history