-
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
minor docs improvements #386
Conversation
B can be repeated in GMT, but I found it confusing to learn how to repeat -B in pygmt (eventually found it in the Note for Experienced GMT Users). This change to the documentation would help clarify the confusion.
I accidentally found that -L works in coast as well as basemap. It would help to have this in the docs.
💖 Thanks for opening this pull request! 💖 Please make sure you read our contributing guidelines and abide by our code of conduct. A few things to keep in mind:
|
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.
Hi @kmaterna, thanks for opening this Pull Request! I just have one minor suggestion (add an alias for L
in coast
), otherwise it looks good. Let me know If there's any other related improvements you'd like to add, and I'll wait for those before merging this PR in 😄
🎉🎉🎉 Congrats on merging your first pull request and welcome to the team! 🎉🎉🎉 Please open a new pull request to add yourself to the |
Thanks @kmaterna! Feel free to point out any other things in the documentation you'd like to improve, or new features you'd like to see in the future. Pull requests like this are always very welcome 😃 |
First ever pull request! I've been using pygmt and it's great- thanks for making it.
I found a few minor points in the docs that would hopefully clarify "coast" and "-B" in my experience.