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 a "community" page with maintainer info #4962

Merged
merged 2 commits into from
Nov 16, 2024
Merged

Conversation

neubig
Copy link
Contributor

@neubig neubig commented Nov 13, 2024

End-user friendly description of the problem this fixes or functionality that this introduces

  • Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

This PR adds a somewhat more organized "community" page, and importantly adds information about how we nominate maintainers


Give a summary of what the PR does, explaining any non-trivial design decisions

The maintainer policy is just a draft/RFP. I'd like maintainers to comment if you can!


Link of any specific issues this addresses

We did not have a clear policy about how maintainers are nominated, but this will fix that.


To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:96d82db-nikolaik   --name openhands-app-96d82db   docker.all-hands.dev/all-hands-ai/openhands:96d82db

tobitege
tobitege previously approved these changes Nov 16, 2024
Copy link
Collaborator

@tobitege tobitege left a comment

Choose a reason for hiding this comment

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

LGTM
This is a very nice addition for transparency!

@tobitege tobitege dismissed their stale review November 16, 2024 06:36

Reconsidered

@enyst
Copy link
Collaborator

enyst commented Nov 16, 2024

Thank you for this! To summarize quickly the underlying issue here:

I think building a startup on this project has been and continues to be a great idea, and I know it is and may still be a little tricky at times to mix and match the concerns of the company with the open source project. This isn't just a matter of license, it is also something to do with attempting to maintain a genuinely community-driven development process where all contributors, whether affiliated with All Hands or elsewhere, operate under the same expectations.

This maintainer nomination policy gets it right, and I really appreciate the care and thought that went into crafting it - and making it real.

@neubig neubig merged commit 104f52b into main Nov 16, 2024
14 checks passed
@neubig neubig deleted the neubig/add_community_md branch November 16, 2024 13:10
@neubig
Copy link
Contributor Author

neubig commented Nov 16, 2024

Thank you @enyst ! And thanks for your help and feedback in drafting it.

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.

3 participants