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

README: Mention that the package is not actively maintained or tested, and add a call for maintainers #224

Merged
merged 2 commits into from
Jan 2, 2025

Conversation

DilumAluthge
Copy link
Member

Alternative to #197.

The difference from #197 is that in this PR, we explicitly say that we have the goal of splitting up this package (#58).

#197 says "looking for a maintainer" - in this PR, we instead explicitly ask only for people to maintain a single manager. I think this is a more realistic goal. I think it's more likely that we can find e.g. three separate people to maintain three separate managers vs. finding a single person willing to maintain all of the managers that currently live in this package.

Copy link
Collaborator

@MilesCranmer MilesCranmer left a comment

Choose a reason for hiding this comment

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

The wording sounds good to me

@DilumAluthge
Copy link
Member Author

I'll merge this now, since I think it's an improvement over the status quo (which is that nothing is mentioned in the README).

I've requested several other people for review. If reviewers have post-merge review comments, I can make follow-up PRs to address them.

@DilumAluthge DilumAluthge merged commit 0c8140b into master Jan 2, 2025
0 of 5 checks passed
@DilumAluthge DilumAluthge deleted the dpa/readme-split-up branch January 2, 2025 06:19
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.

2 participants