-
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
Fix Transverse Mercator documentation typo #2253
Conversation
Co-authored-by: Yvonne Fröhlich <94163266+yvonnefroehlich@users.noreply.github.com>
Hm. Do we plan to have separate PRs for each projection? The PR title was renamed to address only the "Transverse Mercator projection". However, there are some more typos to fix, at least the ones listed at #2059 (comment) 🙂. |
@yvonnefroehlich I had originally planned to fix all of the typos, but I misinterpreted your original comment as asking for someone to fix them. Since I think there needs to be a bit more explanation on what PyGMT typos need fixing (discussed here) vs. what are issues with GMT documentation, it might just be easier if you submitted a PR with your desired fixes. Happy to help, but I don't want to duplicate effort. |
No worries @willschlitzer! Firstly, I wanted only to know how to proceed with @GenericMappingTools/pygmt-maintainers and @willschlitzer Should we leave this PR as is and merge it and open a follow up PR for the remaining typo fixes, or do you prefer closing this PR without merging it and opening a new PR including this one typo fix, to have all these typo fixes together in one PR? |
Either works. Or you can push more commits directly to this PR. |
I prefere working in an own PR on this and starte doing so in PR #2269. |
OK. I think we can close this PR. |
Fixes the Transverse Mercator syntax issues in #2059.
Addresses #2059
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