-
Notifications
You must be signed in to change notification settings - Fork 37
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
Updates Fortran documentation for error codes #284
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great content, just a few minor quibbles. Also, please update the changelog
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #284 +/- ##
===========================================
- Coverage 87.70% 87.58% -0.12%
===========================================
Files 60 60
Lines 3374 3374
===========================================
- Hits 2959 2955 -4
- Misses 415 419 +4 |
I'm going to wait until #285 is merged so we get a clean pass on the CI. |
The examples in the Fortran documentation examples used a mixture of subroutines and functions. The former was deprecated when error handling was implemented to allow the clients to bubble up a return code that Fortran applications can check. This has now been updated to allow others to use the documents as an accurate jumping off point.
Grep for and looking for that string
4d547f4
to
db7032d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking care of these updates! This will help our users getting started with SmartRedis in Fortran quite a bit
The examples in the Fortran documentation examples used a mixture of subroutines and functions. The former was deprecated when error handling was implemented to allow the clients to bubble up a return code that Fortran applications can check. This has now been updated to allow others to use the documents as an accurate jumping off point.
Closes CrayLabs/SmartRedis#335