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

DM-14001: Make afw pep8 compliant and enable auto testing #341

Merged
merged 8 commits into from
Apr 5, 2018
Merged

DM-14001: Make afw pep8 compliant and enable auto testing #341

merged 8 commits into from
Apr 5, 2018

Commits on Apr 5, 2018

  1. Fix pep8 errors

    @r-owen
    r-owen committed Apr 5, 2018
    Configuration menu
    Copy the full SHA
    a99552a View commit details
    Browse the repository at this point in the history

  2. Remove trailing + from concatenated strings 

    A trailing space is not needed when concatenating two string
constants.
    @r-owen
    r-owen committed Apr 5, 2018
    Configuration menu
    Copy the full SHA
    b741342 View commit details
    Browse the repository at this point in the history

  3. Enable automatic running of flake8

    @r-owen
    r-owen committed Apr 5, 2018
    Configuration menu
    Copy the full SHA
    acf67ea View commit details
    Browse the repository at this point in the history

  4. Add Travis flake8 testing

    @r-owen
    r-owen committed Apr 5, 2018
    Configuration menu
    Copy the full SHA
    4f677fb View commit details
    Browse the repository at this point in the history

  5. Fix a pytest warning 

    testGetNonContiguous in test_sourceTable.py used
self.assertFloatsEqual to compare two bool arrays;
use np.testing.assert_equal instead.
    @r-owen
    r-owen committed Apr 5, 2018
    Configuration menu
    Copy the full SHA
    c4d42d4 View commit details
    Browse the repository at this point in the history

  6. Fix Doxygen duplicate section name errors

    @r-owen
    r-owen committed Apr 5, 2018
    Configuration menu
    Copy the full SHA
    74b98c8 View commit details
    Browse the repository at this point in the history

  7. Stop Doxygen from scanning C++ source files 

    We long ago moved all Doxygen documentation out of .cc files
into .h files. Stop scanning the .cc files, as it slows down
Doxygen and is a frequent source of warnings.
    @r-owen
    r-owen committed Apr 5, 2018
    Configuration menu
    Copy the full SHA
    2534050 View commit details
    Browse the repository at this point in the history

  8. Make doc for a private function not doxygen

    @r-owen
    r-owen committed Apr 5, 2018
    Configuration menu
    Copy the full SHA
    4adda76 View commit details
    Browse the repository at this point in the history