Skip to content

Cleanup and standardize badge colors in groups - #4776

Merged
seisman merged 5 commits into
mainfrom
badges
Aug 1, 2026
Merged

Cleanup and standardize badge colors in groups#4776
seisman merged 5 commits into
mainfrom
badges

Conversation

@seisman

@seisman seisman commented Jul 29, 2026

Copy link
Copy Markdown
Member

Before
image

After

image

Updated badges:

  • Compatible Python versions => PyPI Python Version
  • Update "DOI" to "Zenodo DOI". The "Zenodo DOI badge" reflects the concept DOI, not the DOI of the latest version, so it belongs to the 2nd group.

Newly added badges:

  • GitHub Release
  • PyGMT paper

Removed badges:

  • Discourse forum
  • PyOpenSci
  • Ruff
  • CodSpeed Performance Benchmarks
  • Contributor Code of Conduct

Closes #4768

Preview: https://github.com/GenericMappingTools/pygmt/blob/badges/README.md

@seisman seisman added maintenance Boring but important stuff for the core devs needs review This PR has higher priority and needs review. labels Jul 29, 2026
@seisman seisman added this to the 0.20.0 milestone Jul 29, 2026
Comment thread README.md Outdated

@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.

Realy like the sorting of the badge by color.

Comment thread README.md
@@ -2,19 +2,19 @@

[Documentation (development version)](https://www.pygmt.org/dev) | [Contact](https://forum.generic-mapping-tools.org) | [TryOnline](https://github.com/GenericMappingTools/try-gmt)

@yvonnefroehlich yvonnefroehlich Jul 29, 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.

Could we change "Contact" into "GMT Forum"?
(As this is not directly related to the badges, also fine with including this in another PR or in #4778.)

Suggested change
[Documentation (development version)](https://www.pygmt.org/dev) | [Contact](https://forum.generic-mapping-tools.org) | [TryOnline](https://github.com/GenericMappingTools/try-gmt)
[Documentation (development version)](https://www.pygmt.org/dev) | [GMT Forum](https://forum.generic-mapping-tools.org) | [TryOnline](https://github.com/GenericMappingTools/try-gmt)

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.

See my comment https://github.com/GenericMappingTools/pygmt/pull/4776/changes#r3679209568. If we decide to remove the "Documentation (development version)" link, we may also want to remove the "Contact" and "TryOnline" link. So, let's revisit it separately.

Comment thread README.md
@@ -2,19 +2,19 @@

[Documentation (development version)](https://www.pygmt.org/dev) | [Contact](https://forum.generic-mapping-tools.org) | [TryOnline](https://github.com/GenericMappingTools/try-gmt)

@yvonnefroehlich yvonnefroehlich Jul 29, 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.

I am wondering if it makes more sense to link to the latest version of the documentation, as it's likely not too many users install the dev version and update it over time.
(As this is not directly related to the badges, also fine with including this in another PR or in #4778.)

Suggested change
[Documentation (development version)](https://www.pygmt.org/dev) | [Contact](https://forum.generic-mapping-tools.org) | [TryOnline](https://github.com/GenericMappingTools/try-gmt)
[Documentation](https://www.pygmt.org/latest) | [Contact](https://forum.generic-mapping-tools.org) | [TryOnline](https://github.com/GenericMappingTools/try-gmt)

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.

As mentioned in #4746 (comment), I have proposed to add a "Documentation" section in README.md, after which I feel the link is not longer necessary. So Let's revisit it later.

@seisman seisman added final review call This PR requires final review and approval from a second reviewer skip-changelog Skip adding Pull Request to changelog and removed needs review This PR has higher priority and needs review. final review call This PR requires final review and approval from a second reviewer labels Jul 30, 2026
@seisman
seisman merged commit 3054354 into main Aug 1, 2026
12 checks passed
@seisman
seisman deleted the badges branch August 1, 2026 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

Reorganize badges in the README?

2 participants