-
Notifications
You must be signed in to change notification settings - Fork 224
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
Add inline example for grdview #2381
Conversation
Ping @willschlitzer to see if you have time to finish this PR before the v0.9.0 release. |
Co-authored-by: Michael Grund <[email protected]> Co-authored-by: Yvonne Fröhlich <[email protected]>
It's likely that Will is busy recently, so I'm taking over this PR to make sure that we can have it done before v0.9.0. |
Co-authored-by: Yvonne Fröhlich <[email protected]>
Hey, sorry that I've been MIA; I should have time to work on things this week, but no problem with @seisman making changes/approving things. |
Don't worry. Welcome back. |
Co-authored-by: Yvonne Fröhlich <[email protected]>
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.
Hm. Just wondering whether we should decide about starting comments with either upper-case letters or with lower-case letters, across all examples? I have a little preference towards upper-case letters, but I am also totally fine with the lower-case option 🙂.
Either is fine to me. |
Addresses #1957
Reminders
make format
andmake check
to make sure the code follows the style guide.doc/api/index.rst
.Slash Commands
You can write slash commands (
/command
) in the first line of a comment to performspecific operations. Supported slash commands are:
/format
: automatically format and lint the code/test-gmt-dev
: run full tests on the latest GMT development version