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

Fix some lgtm alerts #16613

Merged
merged 8 commits into from
Jun 7, 2017
Merged

Fix some lgtm alerts #16613

merged 8 commits into from
Jun 7, 2017

Commits on Jun 6, 2017

  1. fix unreachable exception

    @jhelie
    jhelie committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    ea90033 View commit details
    Browse the repository at this point in the history

  2. remove unreachable return

    @jhelie
    jhelie committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    4353719 View commit details
    Browse the repository at this point in the history

  3. fix argument name in call

    @jhelie
    jhelie committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    8419332 View commit details
    Browse the repository at this point in the history

  4. fix argument name in formatting call, useless assignment and wrong ex… 

    …ception raised
    @jhelie
    jhelie committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    8fce8d3 View commit details
    Browse the repository at this point in the history

  5. remove useless formatting argument

    @jhelie
    jhelie committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    91bc03f View commit details
    Browse the repository at this point in the history

  6. fix argument name in call

    @jhelie
    jhelie committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    a35dc36 View commit details
    Browse the repository at this point in the history

  7. fix 2 useless assignments to self

    @jhelie
    jhelie committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    116077e View commit details
    Browse the repository at this point in the history

  8. fix test: respect PEP8 format

    @jhelie
    jhelie committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    eecbbb8 View commit details
    Browse the repository at this point in the history