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.

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

Clearer flowline model code and new diagnostics #853

Merged
merged 5 commits into from Aug 15, 2019

Conversation

@fmaussion
Copy link
Member

commented Aug 14, 2019

  • Tests added/passed
  • Fully documented
  • Entry in whats-new.rst
@pep8speaks

This comment has been minimized.

Copy link

commented Aug 14, 2019

Hello @fmaussion! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2019-08-15 10:04:37 UTC
fmaussion added 3 commits Aug 14, 2019

@fmaussion fmaussion merged commit 503b563 into OGGM:master Aug 15, 2019

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@fmaussion fmaussion referenced this pull request Aug 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.