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

PR: Refactoring and adding tests for the brf module. #132

Merged
merged 25 commits into from
Dec 22, 2017
Merged

PR: Refactoring and adding tests for the brf module. #132

merged 25 commits into from
Dec 22, 2017

Commits on Dec 21, 2017

  1. Clean and Make def installation dir more robust

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    21f632d View commit details
    Browse the repository at this point in the history

  2. Add back the water level dataset at the end of the test.

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    6edc368 View commit details
    Browse the repository at this point in the history

  3. Import matplotlib in mainwindow and specify backend

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    7dd29ba View commit details
    Browse the repository at this point in the history

  4. Test the calculation and plotting of the brf

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    dca39dc View commit details
    Browse the repository at this point in the history

  5. Added a docstring to BRFViewer

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    819fcb8 View commit details
    Browse the repository at this point in the history

  6. pep8 corrections

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    6c77726 View commit details
    Browse the repository at this point in the history

  7. Trying to run kgs_brf on Travis with wine

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    4bf70ce View commit details
    Browse the repository at this point in the history

  8. Install Wine correctly on Travis

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    5973c9d View commit details
    Browse the repository at this point in the history

  9. Trying to install Wine on Travis try#2

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    1498e73 View commit details
    Browse the repository at this point in the history

  10. Trying to run kgs_brf on linux try#3

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    24c3f80 View commit details
    Browse the repository at this point in the history

  11. Trying to run kgs_brf on Travis try#4

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    3527874 View commit details
    Browse the repository at this point in the history

  12. Skip test on Linux 

    Until I can debug this on a real machine, I'm going to skip this test.
    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    fa5e22e View commit details
    Browse the repository at this point in the history

  13. Added a reason to skipif

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    30821b4 View commit details
    Browse the repository at this point in the history

  14. Reverted an un-needed change

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    0b62865 View commit details
    Browse the repository at this point in the history

  15. Removed commented code

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    fb7f42d View commit details
    Browse the repository at this point in the history

  16. Trying on Travis again

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    8709a8e View commit details
    Browse the repository at this point in the history

  17. Another try

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    4583ce5 View commit details
    Browse the repository at this point in the history

  18. Skip test on Linux

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    4507653 View commit details
    Browse the repository at this point in the history

  19. Report to coveralls only for AppVeyor

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    8d1e2dd View commit details
    Browse the repository at this point in the history

  20. Reverted changes to runtest.

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    2a60f65 View commit details
    Browse the repository at this point in the history

  21. Fix error instead of skipping test

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    80d862d View commit details
    Browse the repository at this point in the history

  22. Prevent install instead of preventing running on Linux

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    f88523c View commit details
    Browse the repository at this point in the history

  23. Make the test pass in Linux

    @jnsebgosselin
    jnsebgosselin committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    312f62d View commit details
    Browse the repository at this point in the history

Commits on Dec 22, 2017

  1. Commented coveralls report for Travis

    @jnsebgosselin
    jnsebgosselin committed Dec 22, 2017
    Configuration menu
    Copy the full SHA
    13d020e View commit details
    Browse the repository at this point in the history

  2. Corrected small typo in the OS not supported msg

    @jnsebgosselin
    jnsebgosselin committed Dec 22, 2017
    Configuration menu
    Copy the full SHA
    9201bf0 View commit details
    Browse the repository at this point in the history