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

Maintainer announcement update (to README, etc.) #1795

Merged
merged 2 commits into from
Jul 6, 2023

Conversation

sukritsingh
Copy link
Collaborator

Starting a PR where I'm updating the MDTraj README to indicate the transition to new maintainers (and welcoming continued support/contributions from folks). Thoughts/edits/discussions welcome!

For now at least the README is the only change made. I may have to dig around the docs a bit to ensure that appropriate contact info is placed there as well.

@sukritsingh
Copy link
Collaborator Author

@mattwthompson you may have thoughts here? Happy to add you or something since you've been so active as of late

@sukritsingh sukritsingh changed the title Maintainer announcement update (to README, etc.) [WIP] Maintainer announcement update (to README, etc.) Jul 6, 2023
Copy link
Member

@mattwthompson mattwthompson left a comment

Choose a reason for hiding this comment

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

Great news! Is this directly funded, or just sorta swinging under the umbrella of F@H (and I assume the hand-off came from @rmcgibbo in some capacity)?

These changes look good to me - I don't mind being mentioned and don't really care either way. My only suggestion, which I assume is already the intent, is that the note is removed altogether in a few weeks once the repo gets some TLC.

@sukritsingh
Copy link
Collaborator Author

sukritsingh commented Jul 6, 2023

Sorta swinging under the umbrella of F@H for now - Xuhui is also working on taking over MSMBuilder and bringing that back up to speed so there's sort of a two pronged effort to get these FAH-relevant libraries back to order. I wasn't there for the handoff directly but @xuhuihuang and @gbowman were given access power by @rmcgibbo to my knowledge (and Xuhui gave me powers in turn)!

My only suggestion, which I assume is already the intent, is that the note is removed altogether in a few weeks once the repo gets some TLC.

Hah yes the intention is that we a) put this note up for a bit, b) announce on any FAH social channels and c) remove the note after a couple months once we've cleaned up more.

These changes look good to me - I don't mind being mentioned and don't really care either way.

Whatever you're comfortable with! You've put effort in so whatever you'd like to as appropriate acknowledgement!

README.md Outdated Show resolved Hide resolved
@sukritsingh
Copy link
Collaborator Author

Added you in! I'm going to quickly email Xuhui and Greg to get their eyes on this announcement. I'll merge in 24 hours if nothing else comes up.

@peastman
Copy link
Contributor

peastman commented Jul 6, 2023

My only comment is to say I'm really glad it will be actively maintained again. It's a great library.

@sukritsingh sukritsingh changed the title [WIP] Maintainer announcement update (to README, etc.) Maintainer announcement update (to README, etc.) Jul 6, 2023
@sukritsingh
Copy link
Collaborator Author

Got the go-ahead from both! Merging it now.

@sukritsingh sukritsingh merged commit c0295ae into master Jul 6, 2023
6 checks passed
@rmcgibbo
Copy link
Member

rmcgibbo commented Jul 7, 2023

Cheers all! I’m happy to see this, and to hand off the torch to such a capable crew!

-Robert

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.

None yet

4 participants