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

Update the onboarding access checklist in Maintainers Guides #3404

Merged
merged 5 commits into from
Aug 22, 2024

Conversation

seisman
Copy link
Member

@seisman seisman commented Aug 21, 2024

@@ -19,14 +17,12 @@ communication tools we use.

- Added to the [pygmt-contributors team](https://github.com/orgs/GenericMappingTools/teams/pygmt-contributors) (gives 'write' permission to the repository)
- Added as a collaborator on [DAGsHub](https://dagshub.com/GenericMappingTools/pygmt/settings/collaboration) (gives 'write' permission to dvc remote storage)
- Added to the [PyGMT devs Slack channel](https://pygmtdevs.slack.com) (for casual conversations)
Copy link
Member Author

Choose a reason for hiding this comment

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

I'd like to know if we should sunset the PyGMT devs Slack channel since it's no longer actively used and messages older than 90 days are deleted. We can use GitHub Discussions (not enabled for the PyGMT repository, but see https://github.com/GenericMappingTools/gmt/discussions for an example) for open discussions and emails for confidential discussions.

@GenericMappingTools/pygmt-maintainers Please vote on this comment 👍 or 👎. Or leave your comments.

Copy link
Member

@weiji14 weiji14 Aug 21, 2024

Choose a reason for hiding this comment

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

In favour because we haven't used Slack much and their policy deletion policy is getting worse https://forums.slackcommunity.com/s/question/0D5Hq00009rcK9vKAE/free-workspace-one-year-deletion-policy?language=en_US. We could always send a Discourse forum message, or email if necessary.

Copy link
Member

Choose a reason for hiding this comment

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

I agree on this. Additional to these aspects, I still face a strange behavior of getting notifications by Slack.

@@ -19,14 +17,12 @@ communication tools we use.

- Added to the [pygmt-contributors team](https://github.com/orgs/GenericMappingTools/teams/pygmt-contributors) (gives 'write' permission to the repository)
- Added as a collaborator on [DAGsHub](https://dagshub.com/GenericMappingTools/pygmt/settings/collaboration) (gives 'write' permission to dvc remote storage)
- Added to the [PyGMT devs Slack channel](https://pygmtdevs.slack.com) (for casual conversations)
- Added to the {doc}`Team Gallery page <team>`
Copy link
Member Author

Choose a reason for hiding this comment

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

This entry is removed because, A new contributor usually cannot be recognized as "Distinguished Contributors" and is not qualified to be added to the "PyGMT team page".

@seisman seisman added the maintenance Boring but important stuff for the core devs label Aug 21, 2024
@seisman seisman added this to the 0.13.0 milestone Aug 21, 2024
@seisman seisman added the needs review This PR has higher priority and needs review. label Aug 21, 2024
@seisman seisman requested a review from a team August 21, 2024 01:13
doc/maintenance.md Outdated Show resolved Hide resolved
@seisman seisman removed the needs review This PR has higher priority and needs review. label Aug 22, 2024
@seisman seisman merged commit e2019db into main Aug 22, 2024
8 of 9 checks passed
@seisman seisman deleted the doc/maintainers branch August 22, 2024 05:19
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants