Skip to content
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

Update gmtmath.rst #7102

Closed
wants to merge 1 commit into from
Closed

Update gmtmath.rst #7102

wants to merge 1 commit into from

Conversation

Esteban82
Copy link
Member

Update the table operator as discused on #6778.

See #6778 for details.
@Esteban82 Esteban82 added the documentation Improve documentation label Nov 21, 2022
@Esteban82 Esteban82 self-assigned this Nov 21, 2022
@seisman
Copy link
Member

seisman commented Nov 21, 2022

I don't check any changes in this PR, but I think perhaps we should use the "Simple Table" syntax (https://docutils.sourceforge.io/docs/ref/rst/restructuredtext.html#simple-tables) or the "List Table" syntax (https://docutils.sourceforge.io/docs/ref/rst/directives.html#list-table).

@Esteban82
Copy link
Member Author

Ok. I will check how to make the table with those formats. Which one do you prefer?

@seisman
Copy link
Member

seisman commented Nov 23, 2022

I think "Simple Table" is better in this case.

@Esteban82
Copy link
Member Author

I think it is better to close this PR and open a new one with a "Simple Table".

@PaulWessel
Copy link
Member

Fine with me - whatever you think looks better.

@seisman
Copy link
Member

seisman commented Nov 24, 2022

I think it is better to close this PR and open a new one with a "Simple Table".

Fine with me.

@Esteban82 Esteban82 closed this Nov 25, 2022
@joa-quim joa-quim deleted the gmtmath_improveTable branch April 12, 2024 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improve documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants