Skip to content

DOC: Add a page for common parameters in the Technical Reference section#3844

Merged
seisman merged 5 commits intomainfrom
doc/common-parameters
Mar 24, 2025
Merged

DOC: Add a page for common parameters in the Technical Reference section#3844
seisman merged 5 commits intomainfrom
doc/common-parameters

Conversation

@seisman
Copy link
Member

@seisman seisman commented Mar 11, 2025

@seisman seisman added documentation Improvements or additions to documentation needs review This PR has higher priority and needs review. labels Mar 18, 2025
@seisman seisman added this to the 0.15.0 milestone Mar 18, 2025
@seisman seisman added the skip-changelog Skip adding Pull Request to changelog label Mar 18, 2025
@seisman seisman changed the title POC: DOC: Add a page for common parameters in the Technical Reference section DOC: Add a page for common parameters in the Technical Reference section Mar 18, 2025
@seisman seisman marked this pull request as ready for review March 18, 2025 03:59
@seisman seisman requested a review from Copilot March 18, 2025 04:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

The purpose of this PR is to add a new documentation page for common parameters in the Technical Reference section, addressing issue #3843.

  • Added a common parameters page with a glossary block for documenting the "verbose" parameter.
  • Updated the Technical Reference toctree to include the new page.
  • Simplified the verbose parameter description in the decorators module to reference the full usage documentation.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
doc/techref/common_parameters.md Added new documentation page for common parameters.
doc/techref/index.md Updated toctree to include the new page.
pygmt/helpers/decorators.py Updated verbose parameter docstring to link to the full usage.

@michaelgrund michaelgrund added final review call This PR requires final review and approval from a second reviewer and removed needs review This PR has higher priority and needs review. labels Mar 24, 2025
@seisman seisman removed the final review call This PR requires final review and approval from a second reviewer label Mar 24, 2025
@seisman seisman merged commit bf00b6b into main Mar 24, 2025
35 checks passed
@seisman seisman deleted the doc/common-parameters branch March 24, 2025 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation skip-changelog Skip adding Pull Request to changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants