Figure.makecpt: Improve the docstring of the 'reverse' parameter

[yvonnefroehlich]

Description of proposed changes

I feel it is fair to mention in the docstring that using the reverse parameter of Figure.makecpt also exchanges the background (B) and foreground (F) colors.

Upstream GMT documentation: https://docs.generic-mapping-tools.org/6.5/makecpt.html#i

Reminders

• Run make format and make check to make sure the code follows the style guide.
• Add tests for new features or tests that would have caught the bug that you're fixing.
• Add new public functions/methods/classes to doc/api/index.rst.
• Write detailed docstrings for all functions/methods.
• If wrapping a new module, open a 'Wrap new GMT module' issue and submit reasonably-sized PRs.
• If adding new functionality, add an example to docstrings or tutorials.
• Use underscores (not hyphens) in names of Python files and directories.

Slash Commands

You can write slash commands (/command) in the first line of a comment to perform
specific operations. Supported slash command is:

• /format: automatically format and lint the code

[seisman]

I feel it's a good time to make the reverse parameter more Pythonic. Instead of using reverse=True, reverse="c" or reverse="z", we may want to use reverse=True, reverse="colors" and reverse="zvalues" instead.

The long argument names colors and zvalues comes from the upstream PR https://github.com/GenericMappingTools/gmt/pull/8280/files (line 45).

[yvonnefroehlich]

I feel it's a good time to make the reverse parameter more Pythonic. Instead of using reverse=True, reverse="c" or reverse="z", we may want to use reverse=True, reverse="colors" and reverse="zvalues" instead.

The long argument names colors and zvalues comes from the upstream PR https://github.com/GenericMappingTools/gmt/pull/8280/files (line 45).

Sounds fair! Should this be done in this PR or in a separate follow-up PR?

[seisman]

I feel it's a good time to make the reverse parameter more Pythonic. Instead of using reverse=True, reverse="c" or reverse="z", we may want to use reverse=True, reverse="colors" and reverse="zvalues" instead.
The long argument names colors and zvalues comes from the upstream PR https://github.com/GenericMappingTools/gmt/pull/8280/files (line 45).

Sounds fair! Should this be done in this PR or in a separate follow-up PR?

Better to do it in this PR, since the docstring needs rewritten for the Pythonic reverse parameter.

[seisman]

I feel it's a good time to make the reverse parameter more Pythonic. Instead of using reverse=True, reverse="c" or reverse="z", we may want to use reverse=True, reverse="colors" and reverse="zvalues" instead.

The long argument names colors and zvalues comes from the upstream PR https://github.com/GenericMappingTools/gmt/pull/8280/files (line 45).

I prefer to have this PR merged first, and will revisit the issue above later, when we have time focusing on improving the makecpt wrapper.