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 linting by using new maligned linter instead of aligncheck #659

Merged
merged 2 commits into from
Oct 31, 2017
Merged

fix linting by using new maligned linter instead of aligncheck #659

merged 2 commits into from
Oct 31, 2017

Commits on Oct 31, 2017

  1. fix linting by using new maligned linter instead of aligncheck 

    upstream switched the alignment check backend and in doing so fails to run
if the old backend is defined in the config.

also skip alignment linting on a struct we use for byte decoding as we have
no choice in its member order.
    @hsitter
    hsitter committed Oct 31, 2017
    Configuration menu
    Copy the full SHA
    46c2182 View commit details
    Browse the repository at this point in the history

  2. whitelist falke E722 in system/lib.py 

    'E722 do not use bare except' wants us not to use except without type
restriction as it catches everything and the kitchen sink. Since we use
them to catch exceptions in test cases this is intentional as we implement
general purpose error handling on test failure there.
    @hsitter
    hsitter committed Oct 31, 2017
    Configuration menu
    Copy the full SHA
    f43801c View commit details
    Browse the repository at this point in the history