Skip to content

Fix ruff to Ruff#4682

Open
seisman wants to merge 1 commit into
mainfrom
typo/ruff
Open

Fix ruff to Ruff#4682
seisman wants to merge 1 commit into
mainfrom
typo/ruff

Conversation

@seisman

@seisman seisman commented Jun 16, 2026

Copy link
Copy Markdown
Member

As per the official documentation (https://docs.astral.sh/ruff/), the official package name is Ruff, not ruff.

@seisman seisman added this to the 0.19.0 milestone Jun 16, 2026
@seisman seisman added maintenance Boring but important stuff for the core devs skip-changelog Skip adding Pull Request to changelog final review call This PR requires final review and approval from a second reviewer labels Jun 16, 2026

@yvonnefroehlich yvonnefroehlich left a comment

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.

We should also fix this in Acknowledgments of the manuscript either during the second of review or when checking the proofs.

Comment thread doc/changes.md
Comment on lines 578 to +580
* Bump to ghostscript 10.03.0 ([#3112](https://github.com/GenericMappingTools/pygmt/pull/3112))
* Bump to ruff 0.3.0 ([#3081](https://github.com/GenericMappingTools/pygmt/pull/3081))
* Enable ruff's PTH (flake8-use-pathlib) rules and fix violations ([#3129](https://github.com/GenericMappingTools/pygmt/pull/3129))
* Bump to Ruff 0.3.0 ([#3081](https://github.com/GenericMappingTools/pygmt/pull/3081))
* Enable Ruff's PTH (flake8-use-pathlib) rules and fix violations ([#3129](https://github.com/GenericMappingTools/pygmt/pull/3129))

@weiji14 weiji14 Jun 16, 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.

Probably don't need to capitalize if it's written like <package> <version> (so use ruff 0.3.0 here)? E.g. we don't capitalize ghostscript on the line above before its version number?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

final review call This PR requires final review and approval from a second reviewer maintenance Boring but important stuff for the core devs skip-changelog Skip adding Pull Request to changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants