Skip to content

Implement open-source maintainer playbook#349

Open
beets wants to merge 3 commits into
datacommonsorg:mainfrom
beets:dataweaver
Open

Implement open-source maintainer playbook#349
beets wants to merge 3 commits into
datacommonsorg:mainfrom
beets:dataweaver

Conversation

@beets
Copy link
Copy Markdown
Contributor

@beets beets commented May 14, 2026

Implementing the suggestions from the playbook. LMK if there are updates we should make.

@beets beets requested review from dhotchkiss and dwnoble May 14, 2026 23:00
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the project's contribution and issue management documentation. It introduces a comprehensive CONTRIBUTING.md that outlines governance, service levels, and a formal RFC process for large features. Additionally, it adds a structured pull request template and streamlines the existing bug report and feature request templates. A review comment identified an incomplete sentence in the updated CONTRIBUTING.md file that requires correction.

Comment thread CONTRIBUTING.md Outdated
@@ -13,16 +41,10 @@ your current agreements on file or to sign a new one.

You generally only need to submit a CLA once, so if you've already submitted one
(even if it was for a different project), you probably don't need to do it
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The sentence in the Contributor License Agreement section is incomplete. It should end with 'again.' to be grammatically correct and consistent with the previous version.

Suggested change
(even if it was for a different project), you probably don't need to do it
(even if it was for a different project), you probably don't need to do it again.

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.

1 participant