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

Add PyGMT #76

Merged
merged 1 commit into from
Sep 8, 2022
Merged

Add PyGMT #76

merged 1 commit into from
Sep 8, 2022

Conversation

weiji14
Copy link
Contributor

@weiji14 weiji14 commented Sep 6, 2022

A Python interface for the Generic Mapping Tools.

Xref pyOpenSci/software-submission#43

@@ -46,3 +46,7 @@
description: 'A phenotyping pipeline for Python.'
maintainer: ["Moritz Lürig"]
link: "https://github.com/mluerig/phenopype"
- package-name: pygmt
description: 'A Python interface for the Generic Mapping Tools.'
maintainer: ["Dongdong Tian", "Max Jones", "Michael Grund", "Wei Ji Leong", "Will Schlitzer"]
Copy link
Contributor Author

@weiji14 weiji14 Sep 6, 2022

Choose a reason for hiding this comment

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

Cc @seisman, @maxrjones, @michaelgrund, @willschlitzer to check if you're ok with having your names added here. Can do a 👍 or 👎

Copy link
Member

Choose a reason for hiding this comment

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

thank you for this @weiji14 @seisman @maxrjones @michaelgrund @willschlitzer please let us know that you are comfortable with being listed as a maintainer here. we will then merge this PR.

Note that not all of this information is displaying currently on the website but I will be working on that over the next months to improve package and maintainer visibility. We will also post about these packages and highlight them in the future as well.

Copy link

@michaelgrund michaelgrund Sep 7, 2022

Choose a reason for hiding this comment

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

Fine for me.

Copy link

Choose a reason for hiding this comment

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

Fine for me.

Choose a reason for hiding this comment

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

That's fine for me to be listed

Copy link
Member

Choose a reason for hiding this comment

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

thank you everyone! @weiji14 i think there are 4 thumbs up there . are you comfortable with my merging this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh yes please, go for it!

Copy link
Member

Choose a reason for hiding this comment

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

wonderful. thank you!! merging!

@lwasser lwasser merged commit 53b628c into pyOpenSci:main Sep 8, 2022
@weiji14 weiji14 deleted the pygmt branch September 8, 2022 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants