Skip to content

DOC: List all dependencies in the installation guides - #4807

Merged
seisman merged 3 commits into
mainfrom
doc/install
Aug 13, 2026
Merged

DOC: List all dependencies in the installation guides#4807
seisman merged 3 commits into
mainfrom
doc/install

Conversation

@seisman

@seisman seisman commented Aug 9, 2026

Copy link
Copy Markdown
Member

@seisman seisman added documentation Improvements or additions to documentation skip-changelog Skip adding Pull Request to changelog needs review This PR has higher priority and needs review. labels Aug 9, 2026
@seisman seisman added this to the 0.20.0 milestone Aug 9, 2026
Comment thread doc/install.md
We recommend following the instructions further on to install GMT 6.
We recommend following the instructions further on to install GMT.

## Dependencies

@yvonnefroehlich yvonnefroehlich Aug 9, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe move the "Dpendencies" section to the top, similar as in the README? To let users know which packages will be installed together with PyGMT.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I prefer to keep the "Quickstart" section as the first section, so that users can install PyGMT without knowing any details.

README is mainly for describing the project, so I think in that file, it makes more sense to have "Dependencies" before "QuickStart".

@seisman
seisman requested review from yvonnefroehlich and a lite review from Copilot August 10, 2026 05:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the installation guide to include an explicit “Dependencies” section (mirroring the README) so users can see required vs optional Python dependencies alongside GMT.

Changes:

  • Reformats parts of doc/install.md for improved readability/line wrapping.
  • Reworks the “Dependencies” section into Required/Optional dependency lists and links to minimum version requirements.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread doc/install.md Outdated
@seisman seisman 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 Aug 12, 2026
@seisman
seisman merged commit 1d2d4f2 into main Aug 13, 2026
10 of 11 checks passed
@seisman
seisman deleted the doc/install branch August 13, 2026 13:26
@seisman seisman changed the title DOC: List all dependencies in the instalation guides DOC: List all dependencies in the installation guides Aug 13, 2026
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 final review call This PR requires final review and approval from a second reviewer skip-changelog Skip adding Pull Request to changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants