Add description for the "columns" arguments

[michaelgrund]

Add missing description for the columns argument in Figure.plot.

Closes #764.

[welcome]

💖 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:

• If you need help writing tests, take a look at the existing ones for inspiration. If you don't know where to start, let us know and we'll walk you through it.
• All new features should be documented. It helps to write the docstrings for your functions/classes before writing the code. This will help you think about your code design and results in better code.
• No matter what, we are really grateful that you put in the effort to do this! 🎉

[michaelgrund]

/format

[seisman]

/format

[seisman]

@michaelgrund Thanks for your PR.

Your /format command doesn't work due to a bug (#767), and I will try to fix it later.

I just made two tiny comments about your changes. Besides that, you also need to manually limit the docstrings to no more than 80 characters per line to fix these styling issues:

flake8 pygmt setup.py doc/conf.py
pygmt/base_plotting.py:786:80: W505 doc line too long (81 > 79 characters)
pygmt/base_plotting.py:787:80: W505 doc line too long (85 > 79 characters)
pygmt/base_plotting.py:788:80: W505 doc line too long (82 > 79 characters)
make: *** [check] Error 1
Makefile:44: recipe for target 'check' failed

[michaelgrund]

@michaelgrund Thanks for your PR.

Your /format command doesn't work due to a bug (#767), and I will try to fix it later.

I just made two tiny comments about your changes. Besides that, you also need to manually limit the docstrings to no more than 80 characters per line to fix these styling issues:

flake8 pygmt setup.py doc/conf.py
pygmt/base_plotting.py:786:80: W505 doc line too long (81 > 79 characters)
pygmt/base_plotting.py:787:80: W505 doc line too long (85 > 79 characters)
pygmt/base_plotting.py:788:80: W505 doc line too long (82 > 79 characters)
make: *** [check] Error 1
Makefile:44: recipe for target 'check' failed

Thanks for fixing it. Totally read over docstrings limit, good to know for next time ;)

[seisman]

/format

[welcome]

🎉🎉🎉 Congrats on merging your first pull request and welcome to the team! 🎉🎉🎉

Please open a new pull request to add yourself to the AUTHORS.md file. We hope that this was a good experience for you. Let us know if there is any way that the contributing process could be improved.