Skip to content

Add initial GOVERNANCE.md#169

Open
Tim Hockin (thockin) wants to merge 1 commit into
agent-substrate:mainfrom
thockin:add-GOVERNANCE
Open

Add initial GOVERNANCE.md#169
Tim Hockin (thockin) wants to merge 1 commit into
agent-substrate:mainfrom
thockin:add-GOVERNANCE

Conversation

@thockin
Copy link
Copy Markdown
Collaborator

Taking over #11 - mostly I just tweaked Davanum Srinivas (@dims) work

@dims
Copy link
Copy Markdown
Collaborator

Looks good, reads well, internally consistent :)

Comment thread GOVERNANCE.md Outdated
Comment thread GOVERNANCE.md
Comment thread GOVERNANCE.md
Comment thread GOVERNANCE.md
Comment on lines +68 to +69
Open a PR. Allow at least one week for discussion. Requires Maintainer approval
to merge.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Not sure if you want to mention if there is different opinion among maitainers, how it should be resolved?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I'm trying to not be TOO prescriptive until there's really a need. I am a big believer is (near-)consensus when possible. Do you think it's critcal just yet?

@linsun
Copy link
Copy Markdown

lgtm overall, only left a small comment regarding updating the governance.

@thockin
Copy link
Copy Markdown
Collaborator Author

Fixed most comments

@ameukam
Copy link
Copy Markdown
Contributor

FWIW, LGTM!

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.

6 participants